report abstract as extraneous modifier

pkg/front_end/lib/src/fasta/parser/parser.dart

Index: pkg/front_end/lib/src/fasta/parser/parser.dart
diff --git a/pkg/front_end/lib/src/fasta/parser/parser.dart b/pkg/front_end/lib/src/fasta/parser/parser.dart
index d3bcceae05a50ab1bdd7ec486e4fa479814c7fa3..2dfaf912f8e08f9338935051cbce9fb982747be9 100644
--- a/pkg/front_end/lib/src/fasta/parser/parser.dart
+++ b/pkg/front_end/lib/src/fasta/parser/parser.dart
@@ -2059,7 +2059,6 @@ class Parser {
       }
       int order = modifierOrder(token);
       if (order < 3) {
-        // `abstract` isn't parsed with this method.
         if (order > currentOrder) {
           currentOrder = order;
           if (optional("var", token)) {
@@ -2149,6 +2148,11 @@ class Parser {
               token, fasta.templateExtraneousModifier);
           token = token.next;
         }
+      } else if (order == 3) {
+        // abstract

[ahe, 2017/08/28 11:14:08]

Consider this instead of the comment:

assert(optional("abstract", token))

[danrubel, 2017/08/29 14:58:47]

Good idea. Done.

+        reportRecoverableErrorWithToken(
+            token, fasta.templateExtraneousModifier);

[ahe, 2017/08/28 11:14:08]

Should this be a new error code so it's easier to distinguish them in the
analyzer?

[danrubel, 2017/08/29 14:58:47]

I need to create a separate error code to distinguish 'abstract' on class
members. I'll address that in a separate CL.

+        token = token.next;
       } else {
         break;
       }