Additional HandleScopes to limit Handle consumption.

src/ast.cc

 // 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.
 
 #include "src/ast.h"
 
 #include <cmath>  // For isfinite.
 #include "src/builtins.h"
 #include "src/code-stubs.h"
 #include "src/contexts.h"
@@ skipping 523 matching lines @@
     Expression* element = values()->at(array_index);
     if (element->IsSpread()) break;
     MaterializedLiteral* m_literal = element->AsMaterializedLiteral();
     if (m_literal != NULL) {
       m_literal->BuildConstants(isolate);
       if (m_literal->depth() + 1 > depth_acc) {
         depth_acc = m_literal->depth() + 1;
       }
     }
     Handle<Object> boilerplate_value = GetBoilerplateValue(element, isolate);
-
+    HandleScope scope(isolate);

[Erik Corry Chromium.org, 2015/06/15 09:42:58]

Please add comment explaining why this can't be moved to the top of the for()
loop.

[oth, 2015/06/15 13:34:03]

Done.

     if (boilerplate_value->IsTheHole()) {
       is_holey = true;
       continue;
     }
 
     if (boilerplate_value->IsUninitialized()) {
       boilerplate_value = handle(Smi::FromInt(0), isolate);
       is_simple = false;
     }
 
@@ skipping 603 matching lines @@
 bool Literal::Match(void* literal1, void* literal2) {
   const AstValue* x = static_cast<Literal*>(literal1)->raw_value();
   const AstValue* y = static_cast<Literal*>(literal2)->raw_value();
   return (x->IsString() && y->IsString() && x->AsString() == y->AsString()) ||
          (x->IsNumber() && y->IsNumber() && x->AsNumber() == y->AsNumber());
 }
 
 
 }  // namespace internal
 }  // namespace v8