Add AstBuilder support for DottedName and ConditionalUri.

pkg/front_end/lib/src/fasta/analyzer/ast_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.analyzer.ast_builder;
 
 import 'package:front_end/src/fasta/scanner/token.dart'
     show BeginGroupToken, Token;
 
 import 'package:analyzer/analyzer.dart';
@@ skipping 682 matching lines @@
         beginToken, scriptTag, directives, declarations, endToken));
   }
 
   void endImport(Token importKeyword, Token deferredKeyword, Token asKeyword,
       Token semicolon) {
     debugEvent("Import");
     List<Combinator> combinators = pop();
     SimpleIdentifier prefix;
     if (asKeyword != null) prefix = pop();
     List<Configuration> configurations = pop();
-    assert(configurations == null); // TODO(paulberry)
     StringLiteral uri = pop();
     List<Annotation> metadata = pop();
     assert(metadata == null);
     // TODO(paulberry): capture doc comments.
     Comment comment = null;
     push(ast.importDirective(
         comment,
         metadata,
         toAnalyzerToken(importKeyword),
         uri,
         configurations,
         toAnalyzerToken(deferredKeyword),
         toAnalyzerToken(asKeyword),
         prefix,
         combinators,
         toAnalyzerToken(semicolon)));
   }
 
   void endExport(Token exportKeyword, Token semicolon) {
     debugEvent("Export");
     List<Combinator> combinators = pop();
     List<Configuration> configurations = pop();
-    assert(configurations == null); // TODO(paulberry)
     StringLiteral uri = pop();
     List<Annotation> metadata = pop();
     assert(metadata == null);
     // TODO(paulberry): capture doc comments.
     Comment comment = null;
     push(ast.exportDirective(comment, metadata, toAnalyzerToken(exportKeyword),
         uri, configurations, combinators, toAnalyzerToken(semicolon)));
   }
 
   @override
+  void endDottedName(int count, Token firstIdentifier) {
+    debugEvent("DottedName");
+    List<SimpleIdentifier> components = popList(count);
+    push(ast.dottedName(components));
+  }
+
+  void endConditionalUri(Token ifKeyword, Token equalitySign) {
+    debugEvent("ConditionalUri");
+    StringLiteral libraryUri = pop();
+    // TODO(paulberry,ahe): the parser should report the right paren token to
+    // the listener.

[ahe, 2017/02/22 05:58:53]

That would be ifKeyword.next.endGroup.

+    Token rightParen = null;
+    StringLiteral value;
+    if (equalitySign != null) {
+      value = pop();
+    }
+    DottedName name = pop();
+    // TODO(paulberry,ahe): what if there is no `(` token due to an error in the
+    // file being parsed?  It seems like we need the parser to do adequate error
+    // recovery and then report both the ifKeyword and leftParen tokens to the
+    // listener.
+    Token leftParen = ifKeyword.next;

[ahe, 2017/02/22 05:58:53]

The scanner should have set up the token stream so that there's a synthetic
endGroup token for the right parenthesis.

The next token of the endGroup will be an error token. The parser may not
recover from this situation.

[Paul Berry, 2017/02/22 21:20:13]

I think you're answering the question "what if there is no ')' token due to an
error in the file being parsed?"  My question is "what if there is no '('
token?"

Currently, the parser calls expect() to consume the '(' token, and expect()
calls reportUnrecoverableError(), which calls handleUnrecoverableError(), which
throws an exception.  So the question is moot for now.

But in the long run we're going to have to change things so that when the parser
is invoked by analyzer, the error can be recovered from, at which point we're
going to have to decide what should happen here.  I don't know whether we'll
wind up introducing a synthetic '(' token or doing something else.  We'll have
to make sure completion does the right thing.

Leaving the code as is for now.

[ahe, 2017/02/23 17:51:52]

Yes. I answered "what if there's no close parenthesis".
 
Yes, it is moot for now.

However, handleUnrecoverableError may recover. Take a look at handleError in
pkg/compiler/lib/src/parser/element_listener.dart.
I have been thinking that one should start by introducing a synthetic version of
whatever is missing, however, it's more complicated than that.
Yes. I'm just using your todos as an opportunity to provide some additional
details.

+    push(ast.configuration(
+        toAnalyzerToken(ifKeyword),
+        toAnalyzerToken(leftParen),
+        name,
+        toAnalyzerToken(equalitySign),
+        value,
+        toAnalyzerToken(rightParen),
+        libraryUri));
+  }
+
+  @override
   void endConditionalUris(int count) {
     debugEvent("ConditionalUris");
     push(popList(count) ?? NullValue.ConditionalUris);
   }
 
   @override
   void endIdentifierList(int count) {
     debugEvent("IdentifierList");
     push(popList(count) ?? NullValue.IdentifierList);
   }
@@ skipping 375 matching lines @@
 /// [ClassDeclaration] or [ClassTypeAlias] object.
 class _MixinApplication {
   final TypeName supertype;
 
   final Token withKeyword;
 
   final List<TypeName> mixinTypes;
 
   _MixinApplication(this.supertype, this.withKeyword, this.mixinTypes);
 }