Make the Isolate parameter mandatory for internal HandleScopes.

src/elements.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 465 matching lines @@
     int entry = from->FindEntry(i + from_start);
     if (entry != SeededNumberDictionary::kNotFound) {
       to->set(i + to_start, from->ValueAt(entry)->Number());
     } else {
       to->set_the_hole(i + to_start);
     }
   }
 }
 
 
-static void TraceTopFrame() {
-  StackFrameIterator it;
+static void TraceTopFrame(Isolate* isolate) {
+  StackFrameIterator it(isolate);
   if (it.done()) {
     PrintF("unknown location (no JavaScript frames present)");
     return;
   }
   StackFrame* raw_frame = it.frame();
   if (raw_frame->is_internal()) {
     Isolate* isolate = Isolate::Current();
     Code* apply_builtin = isolate->builtins()->builtin(
         Builtins::kFunctionApply);
     if (raw_frame->unchecked_code() == apply_builtin) {
       PrintF("apply from ");
       it.Advance();
       raw_frame = it.frame();
     }
   }
-  JavaScriptFrame::PrintTop(stdout, false, true);
+  JavaScriptFrame::PrintTop(isolate, stdout, false, true);
 }
 
 
 void CheckArrayAbuse(JSObject* obj, const char* op, uint32_t key,
                      bool allow_appending) {
   Object* raw_length = NULL;
   const char* elements_type = "array";
   if (obj->IsJSArray()) {
     JSArray* array = JSArray::cast(obj);
     raw_length = array->length();
   } else {
     raw_length = Smi::FromInt(obj->elements()->length());
     elements_type = "object";
   }
 
   if (raw_length->IsNumber()) {
     double n = raw_length->Number();
     if (FastI2D(FastD2UI(n)) == n) {
       int32_t int32_length = DoubleToInt32(n);
       uint32_t compare_length = static_cast<uint32_t>(int32_length);
       if (allow_appending) compare_length++;
       if (key >= compare_length) {
         PrintF("[OOB %s %s (%s length = %d, element accessed = %d) in ",
                elements_type, op, elements_type,
                static_cast<int>(int32_length),
                static_cast<int>(key));
-        TraceTopFrame();
+        TraceTopFrame(obj->GetIsolate());
         PrintF("]\n");
       }
     } else {
       PrintF("[%s elements length not integer value in ", elements_type);
-      TraceTopFrame();
+      TraceTopFrame(obj->GetIsolate());
       PrintF("]\n");
     }
   } else {
     PrintF("[%s elements length not a number in ", elements_type);
-    TraceTopFrame();
+    TraceTopFrame(obj->GetIsolate());
     PrintF("]\n");
   }
 }
 
 
 // Base class for element handler implementations. Contains the
 // the common logic for objects with different ElementsKinds.
 // Subclasses must specialize method for which the element
 // implementation differs from the base class implementation.
 //
@@ skipping 1416 matching lines @@
   if (!maybe_obj->To(&new_backing_store)) return maybe_obj;
   new_backing_store->set(0, length);
   { MaybeObject* result = array->SetContent(new_backing_store);
     if (result->IsFailure()) return result;
   }
   return array;
 }
 
 
 } }  // namespace v8::internal