Reland: [Parse] ParseInfo owns the parsing Zone.

test/cctest/asmjs/test-asm-typer.cc

Index: test/cctest/asmjs/test-asm-typer.cc
diff --git a/test/cctest/asmjs/test-asm-typer.cc b/test/cctest/asmjs/test-asm-typer.cc
index 892c968d1d3e0ebb48fd036dec23c2aece3e87db..24f8f9b67273d190f73ec496cd052ae4e2675aa5 100644
--- a/test/cctest/asmjs/test-asm-typer.cc
+++ b/test/cctest/asmjs/test-asm-typer.cc
@@ -53,7 +53,7 @@ class AsmTyperHarnessBuilder {
         source_code_(
             factory_->NewStringFromUtf8(CStrVector(source)).ToHandleChecked()),
         script_(factory_->NewScript(source_code_)) {
-    ParseInfo info(zone_, script_);
+    ParseInfo info(script_);

[marja, 2017/01/17 15:39:08]

This is a bit weird. We'll create a Zone here, and ParseInfo will create another
one. And we give ParseInfo an AstValueFactory which is from a different Zone...

But that should work anyway, right?

[rmcilroy, 2017/01/19 12:48:29]

You are right and this was actually an issue. I reworked this test code so that
we use the zone from ParseInfo to create the ast-value-factory.

     info.set_allow_lazy_parsing(false);
     info.set_toplevel(true);
     info.set_ast_value_factory(&ast_value_factory_);