Re-land "Add support for metadata on type variables to Fasta parser." (with fixes).

pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart

Index: pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart b/pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart
index fcc7bd58161686f211e8717a4f3cfe1eee283041..ebf7378957bf44ac85b54810f962a6aac79a5eb6 100644
--- a/pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart
+++ b/pkg/front_end/lib/src/fasta/analyzer/ast_builder.dart
@@ -162,6 +162,17 @@ class AstBuilder extends ScopeListener {
     push(ast.methodInvocation(null, null, null, null, arguments));
   }
 
+  void endMetadata(Token beginToken, Token periodBeforeName, Token endToken) {

[Paul Berry, 2017/03/15 16:14:52]

Note: Konstantin introduced an implementation of this method in
08eba174cc9d5ba028f9f0d53a35f1d2190d834e, so it is no longer needed in this CL.

+    debugEvent("Metadata");
+    ArgumentList arguments = pop();
+    SimpleIdentifier constructorName = popIfNotNull(periodBeforeName);
+    var typeArguments = pop();
+    assert(typeArguments == null); // TODO(paulberry)
+    Identifier name = pop();
+    push(ast.annotation(toAnalyzerToken(beginToken), name,
+        toAnalyzerToken(periodBeforeName), constructorName, arguments));
+  }
+
   void handleIdentifier(Token token, IdentifierContext context) {
     debugEvent("handleIdentifier");
     analyzer.Token analyzerToken = toAnalyzerToken(token);
@@ -1387,9 +1398,8 @@ class AstBuilder extends ScopeListener {
     debugEvent("TypeVariable");
     TypeAnnotation bound = pop();
     SimpleIdentifier name = pop();
-    List<Annotation> metadata = null; // TODO(paulberry)
-    // TODO(paulberry): capture doc comments.  See dartbug.com/28851.
-    Comment comment = null;
+    List<Annotation> metadata = pop();
+    Comment comment = pop();
     push(ast.typeParameter(
         comment, metadata, name, toAnalyzerToken(extendsOrSuper), bound));
   }