Revert of Put Scopes into temporary Zone

src/ast/scopes.h

Index: src/ast/scopes.h
diff --git a/src/ast/scopes.h b/src/ast/scopes.h
index 304b877bfacdf803319401502cea2d43778da90c..9ebc7961c6eb49d8587bea7bdbf93aeeabb74556 100644
--- a/src/ast/scopes.h
+++ b/src/ast/scopes.h
@@ -201,7 +201,6 @@
     // the same name because they may be removed selectively via
     // RemoveUnresolved().
     DCHECK(!already_resolved());
-    DCHECK_EQ(factory->zone(), zone());
     VariableProxy* proxy =
         factory->NewVariableProxy(name, kind, start_position, end_position);
     proxy->set_next_unresolved(unresolved_);
@@ -610,13 +609,6 @@
     return &sloppy_block_function_map_;
   }
 
-  // To be called during parsing. Do just enough scope analysis that we can
-  // discard the Scope for lazily compiled functions. In particular, this
-  // records variables which cannot be resolved inside the Scope (we don't yet
-  // know what they will resolve to since the outer Scopes are incomplete) and
-  // migrates them into migrate_to.
-  void AnalyzePartially(Scope* migrate_to, AstNodeFactory* ast_node_factory);
-
   // ---------------------------------------------------------------------------
   // Debugging.
 
@@ -625,9 +617,6 @@
 
   // Check that the scope has positions assigned.
   void CheckScopePositions();
-
-  // Check that all Scopes in the scope tree use the same Zone.
-  void CheckZones();
 #endif
 
   // ---------------------------------------------------------------------------
@@ -788,22 +777,15 @@
   };
 
   // Lookup a variable reference given by name recursively starting with this
-  // scope, but only until max_outer_scope (if not nullptr). If the code is
-  // executed because of a call to 'eval', the context parameter should be set
-  // to the calling context of 'eval'.
+  // scope. If the code is executed because of a call to 'eval', the context
+  // parameter should be set to the calling context of 'eval'.
   Variable* LookupRecursive(VariableProxy* proxy, BindingKind* binding_kind,
-                            AstNodeFactory* factory,
-                            Scope* max_outer_scope = nullptr);
+                            AstNodeFactory* factory);
   MUST_USE_RESULT
   bool ResolveVariable(ParseInfo* info, VariableProxy* proxy,
                        AstNodeFactory* factory);
   MUST_USE_RESULT
   bool ResolveVariablesRecursively(ParseInfo* info, AstNodeFactory* factory);
-
-  // Tries to resolve local variables inside max_outer_scope; collects those
-  // which cannot be resolved.
-  void CollectUnresolvableLocals(VariableProxy** still_unresolved,
-                                 Scope* max_outer_scope);
 
   // Scope analysis.
   void PropagateScopeInfo(bool outer_scope_calls_sloppy_eval);