new handleInvalidTopLevelDeclaration event in fasta parser

pkg/front_end/lib/src/fasta/kernel/body_builder.dart

 // Copyright (c) 2016, 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.
 
 library fasta.body_builder;
 
 import 'package:kernel/ast.dart'
     hide InvalidExpression, InvalidInitializer, InvalidStatement;
 
 import 'package:kernel/class_hierarchy.dart' show ClassHierarchy;
@@ skipping 388 matching lines @@
         push(wrapInCompileTimeError(
             toValue(expression), fasta.messageExpressionNotMetadata));
       } else {
         push(toValue(expression));
       }
       constantExpressionRequired = savedConstantExpressionRequired;
     }
   }
 
   @override
-  void endMetadataStar(int count, bool forParameter) {

[ahe, 2017/08/07 13:31:00]

Why remove this method?

[danrubel, 2017/08/14 16:43:11]

The superclass has an identical method, so I removed it here.

Based on your suggestion and comments in the superclass, I will push the method
out of the superclass and into the various subclasses.
See https://codereview.chromium.org/2998933002/

-    debugEvent("MetadataStar");
-    push(popList(count) ?? NullValue.Metadata);
-  }
-
-  @override
   void endTopLevelFields(int count, Token beginToken, Token endToken) {
     debugEvent("TopLevelFields");
     push(count);
   }
 
   @override
   void endFields(int count, Token beginToken, Token endToken) {
     debugEvent("Fields");
     push(count);
   }
@@ skipping 3376 matching lines @@
       return AsyncMarker.Async;
     } else {
       assert(identical(starToken.stringValue, "*"));
       return AsyncMarker.AsyncStar;
     }
   } else {
     return unhandled(asyncToken.lexeme, "asyncMarkerFromTokens",
         asyncToken.charOffset, null);
   }
 }