Re-introduce ImportDeclaration to the parser

src/parser.h

Index: src/parser.h
diff --git a/src/parser.h b/src/parser.h
index a65a863793f05f1289fe961464034112e318b5fb..b93713b6c41916da38255b8689b9d3a65af7c4b6 100644
--- a/src/parser.h
+++ b/src/parser.h
@@ -705,7 +705,7 @@ class Parser : public ParserBase<ParserTraits> {
   Statement* ParseStatementListItem(bool* ok);
   void* ParseModule(ZoneList<Statement*>* body, bool* ok);
   Statement* ParseModuleItem(bool* ok);
-  Literal* ParseModuleSpecifier(bool* ok);
+  const AstRawString* ParseModuleSpecifier(bool* ok);
   Statement* ParseImportDeclaration(bool* ok);
   Statement* ParseExportDeclaration(bool* ok);
   Statement* ParseExportDefault(bool* ok);
@@ -713,8 +713,7 @@ class Parser : public ParserBase<ParserTraits> {
                           ZoneList<Scanner::Location>* export_locations,
                           ZoneList<const AstRawString*>* local_names,
                           Scanner::Location* reserved_loc, bool* ok);
-  void* ParseNamedImports(ZoneList<const AstRawString*>* import_names,
-                          ZoneList<const AstRawString*>* local_names, bool* ok);
+  ZoneList<ImportDeclaration*>* ParseNamedImports(int pos, bool* ok);

[rossberg, 2015/02/25 13:55:49]

While you're at it, can you perhaps change the other above functions to _return_
their lists as well? I don't see a good reason why they are created by the
caller.

[arv (Not doing code reviews), 2015/02/25 16:07:25]

The one above has 2 lists as out params.

[adamk, 2015/02/25 21:43:12]

Would be fine changing ParseStatementList and ParseModule (to-be-named
ParseModuleItemList) to return their arguments, though I'd rather do it in a
different patch.

[adamk, 2015/02/25 21:50:46]

Actually, there is a reason for passing in body there: ParseStatementList has
multiple callers, and one of them already has a ZoneList<Statement*> with some
statements in it before passing it in. Not sure this is worth changing for now.

   Statement* ParseStatement(ZoneList<const AstRawString*>* labels, bool* ok);
   Statement* ParseSubStatement(ZoneList<const AstRawString*>* labels, bool* ok);
   Statement* ParseFunctionDeclaration(ZoneList<const AstRawString*>* names,