Added a version of the v8::HandleScope constructor with an Isolate and use that consistently.

test/cctest/test-threads.cc

 // Copyright 2008 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 18 matching lines @@
 
 #include "platform.h"
 #include "isolate.h"
 
 #include "cctest.h"
 
 
 TEST(Preemption) {
   v8::Locker locker(CcTest::default_isolate());
   v8::V8::Initialize();
-  v8::HandleScope scope;
+  v8::HandleScope scope(CcTest::default_isolate());
   v8::Context::Scope context_scope(v8::Context::New());
 
   v8::Locker::StartPreemption(100);
 
   v8::Handle<v8::Script> script = v8::Script::Compile(
       v8::String::New("var count = 0; var obj = new Object(); count++;\n"));
 
   script->Run();
 
   v8::Locker::StopPreemption();
@@ skipping 11 matching lines @@
 };
 
 static Turn turn = FILL_CACHE;
 
 
 class ThreadA : public v8::internal::Thread {
  public:
   ThreadA() : Thread("ThreadA") { }
   void Run() {
     v8::Locker locker(CcTest::default_isolate());
-    v8::HandleScope scope;
+    v8::HandleScope scope(CcTest::default_isolate());
     v8::Context::Scope context_scope(v8::Context::New());
 
     CHECK_EQ(FILL_CACHE, turn);
 
     // Fill String.search cache.
     v8::Handle<v8::Script> script = v8::Script::Compile(
         v8::String::New(
           "for (var i = 0; i < 3; i++) {"
           "  var result = \"a\".search(\"a\");"
           "  if (result != 0) throw \"result: \" + result + \" @\" + i;"
@@ skipping 18 matching lines @@
 
 
 class ThreadB : public v8::internal::Thread {
  public:
   ThreadB() : Thread("ThreadB") { }
   void Run() {
     do {
       {
         v8::Locker locker(CcTest::default_isolate());
         if (turn == CLEAN_CACHE) {
-          v8::HandleScope scope;
+          v8::HandleScope scope(CcTest::default_isolate());
           v8::Context::Scope context_scope(v8::Context::New());
 
           // Clear the caches by forcing major GC.
           HEAP->CollectAllGarbage(v8::internal::Heap::kNoGCFlags);
           turn = SECOND_TIME_FILL_CACHE;
           break;
         }
       }
 
       Thread::YieldCPU();
@@ skipping 78 matching lines @@
     Join();
   }
 };
 
 
 TEST(ThreadJoinSelf) {
   ThreadC thread;
   thread.Start();
   thread.Join();
 }