Put FunctionLiterals into temp_zone too.

src/parsing/parser.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_PARSING_PARSER_H_
 #define V8_PARSING_PARSER_H_
 
 #include "src/allocation.h"
 #include "src/ast/ast.h"
 #include "src/ast/scopes.h"
@@ skipping 913 matching lines @@
     Block* block_;
     const DeclarationDescriptor* descriptor_;
     ZoneList<const AstRawString*>* names_;
     Expression* current_value_;
     int recursion_level_;
     bool* ok_;
 
     DEFINE_AST_VISITOR_MEMBERS_WITHOUT_STACKOVERFLOW()
   };
 
+  // Helper for putting parts of the parse results into a temporary zone when
+  // parsing inner function bodies.
+  class DiscardableZoneScope {

[titzer, 2016/07/19 07:55:30]

AFAICT the main difference between DiscardableZoneScope and
AstNodeFactory::BodyScope is that the former contains one of the latter and also
manipulates the function name inferrer.

I don't really understand why function name inferrer needs to be swapped out.

Also, can this class live in the .cc file?

+   public:
+    DiscardableZoneScope(Parser* parser, Zone* temp_zone, bool use_temp_zone)
+        : ast_node_factory_scope_(parser->factory(), temp_zone, use_temp_zone),
+          fni_(parser->ast_value_factory_, temp_zone),
+          parser_(parser),
+          prev_fni_(parser->fni_) {
+      if (use_temp_zone) {
+        parser_->fni_ = &fni_;
+      }
+    }
+    ~DiscardableZoneScope() { parser_->fni_ = prev_fni_; }
+
+   private:
+    AstNodeFactory::BodyScope ast_node_factory_scope_;
+    FuncNameInferrer fni_;
+    Parser* parser_;
+    FuncNameInferrer* prev_fni_;
+  };
+
   Block* ParseVariableDeclarations(VariableDeclarationContext var_context,
                                    DeclarationParsingResult* parsing_result,
                                    ZoneList<const AstRawString*>* names,
                                    bool* ok);
   Statement* ParseExpressionOrLabelledStatement(
       ZoneList<const AstRawString*>* labels,
       AllowLabelledFunctionStatement allow_function, bool* ok);
   IfStatement* ParseIfStatement(ZoneList<const AstRawString*>* labels,
                                 bool* ok);
   Statement* ParseContinueStatement(bool* ok);
@@ skipping 367 matching lines @@
 
 DoExpression* ParserTraits::ParseDoExpression(bool* ok) {
   return parser_->ParseDoExpression(ok);
 }
 
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_PARSING_PARSER_H_