Keep all type arguments in summaries - dynamic or not.

pkg/analyzer/lib/src/summary/idl.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * This file is an "idl" style description of the summary format.  It
  * contains abstract classes which declare the interface for reading data from
  * summaries.  It is parsed and transformed into code that implements the
  * summary format.
  *
@@ skipping 184 matching lines @@
    * If this [EntityRef] is a reference to a function type whose
    * [FunctionElement] is not in any library (e.g. a function type that was
    * synthesized by a LUB computation), the return type of the function.
    * Otherwise `null`.
    */
   @Id(5)
   EntityRef get syntheticReturnType;
 
   /**
    * If this is an instantiation of a generic type or generic executable, the
-   * type arguments used to instantiate it.  Trailing type arguments of type
-   * `dynamic` are omitted.
+   * type arguments used to instantiate it.  If AST is used to create the

[Paul Berry, 2016/05/06 18:11:38]

IMO, the documentation in this file should specify what happens with AST
summaries, and should not mention what happens with element-based summaries. 
Rationale: the behavior of element-based summaries is a bug caused by the fact
that the element model doesn't allow us to tell the difference between implicit
and explicit type arguments; that bug should be documented in the element-based
summary code (and tests), not here.

So personally I would just say "If this is an instantiation of a generic type or
generic executable, the type arguments used to instantiate it (if any)."

[scheglov, 2016/05/06 19:12:36]

Done.

+   * summary, and the AST does not specify any type arguments, this list is
+   * empty.  If element model is used, this list is not empty.
    */
   @Id(1)
   List<EntityRef> get typeArguments;
 }
 
 /**
  * Enum used to indicate the kind of a name in index.
  */
 enum IndexNameKind {
   /**
@@ skipping 2486 matching lines @@
    */
   @Id(11)
   int get visibleLength;
 
   /**
    * If a local variable, the beginning of the visible range; zero otherwise.
    */
   @Id(12)
   int get visibleOffset;
 }