PreParsing inner functions: Fix declaration-only variables, part 2.

src/parsing/parser.cc

Index: src/parsing/parser.cc
diff --git a/src/parsing/parser.cc b/src/parsing/parser.cc
index 0ae1fb619a70ff1aa8bf224fa5a613ec99ebcea4..69f517d9f963b2dc337574e290c40f5a2214ea30 100644
--- a/src/parsing/parser.cc
+++ b/src/parsing/parser.cc
@@ -124,6 +124,7 @@ class DiscardableZoneScope {
       parser_->zone_ = temp_zone;
       if (parser_->reusable_preparser_ != nullptr) {
         parser_->reusable_preparser_->zone_ = temp_zone;
+        parser_->reusable_preparser_->factory()->set_zone(temp_zone);

[adamk, 2016/10/07 18:35:55]

Seems a little weird to set this zone, but not the one for
ast_value_factory...I'm sure I'm missing something, but what is it?

[marja, 2016/10/07 19:00:27]

Offline discussion:
- AstValueFactories always use the main Zone
- PreParserFactory ctor has a weird signature because it needs to match
AstNodeFactory

       }
     }
   }
@@ -132,6 +133,7 @@ class DiscardableZoneScope {
     parser_->zone_ = prev_zone_;
     if (parser_->reusable_preparser_ != nullptr) {
       parser_->reusable_preparser_->zone_ = prev_zone_;
+      parser_->reusable_preparser_->factory()->set_zone(prev_zone_);
     }
     ast_node_factory_scope_.Reset();
   }