[parser] Only track parsing-mode (and possibly switch to the preparser) in the parser

src/parsing/preparser.h

Index: src/parsing/preparser.h
diff --git a/src/parsing/preparser.h b/src/parsing/preparser.h
index 4eb915cfd60c799698e04036eb7d639aa82d56a9..69f830e7cd2914d32bd28d009bf0f77065881480 100644
--- a/src/parsing/preparser.h
+++ b/src/parsing/preparser.h
@@ -911,8 +911,14 @@ class PreParser : public ParserBase<PreParser> {
       const PreParserFormalParameters& parameters, FunctionKind kind,
       FunctionLiteral::FunctionType function_type, bool* ok);
 
+  // Indicates that we won't switch from the preparser to the preparser; we'll
+  // just stay where we are.
   bool AllowsLazyParsingWithoutUnresolvedVariables() const { return false; }
 
+  // Indicates that we won't switch from the preparser to the preparser; we'll

[vogelheim, 2016/11/07 14:56:17]

Since this just duplicates the comments (because both functions serve a similar
purpose), maybe just write the comment + both declarations in one block?

+  // just stay where we are.
+  Mode mode() { return PARSE_EAGERLY; }
+
   V8_INLINE LazyParsingResult SkipFunction(
       FunctionKind kind, DeclarationScope* function_scope, int* num_parameters,
       int* function_length, bool* has_duplicate_parameters,
@@ -1523,7 +1529,6 @@ PreParserStatementList PreParser::ParseEagerFunctionBody(
     PreParserIdentifier function_name, int pos,
     const PreParserFormalParameters& parameters, FunctionKind kind,
     FunctionLiteral::FunctionType function_type, bool* ok) {
-  ParsingModeScope parsing_mode(this, PARSE_EAGERLY);
   PreParserStatementList result;
 
   Scope* inner_scope = scope();