new handleInvalidTopLevelDeclaration event in fasta parser

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

Index: pkg/front_end/lib/src/fasta/kernel/body_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/body_builder.dart b/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
index 3b300a18aff017cebb6192c3942c3f658345e0ef..8c376d7dde657550bced1e530cf48b466fd5b9cd 100644
--- a/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
@@ -406,12 +406,6 @@ class BodyBuilder extends ScopeListener<JumpTarget> implements BuilderHelper {
   }
 
   @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);