Remove more calls to HandleScope default ctor.

chrome/test/base/v8_unit_test.cc

 // Copyright (c) 2011 The Chromium 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 "chrome/test/base/v8_unit_test.h"
 
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/common/chrome_paths.h"
 
 namespace {
 
 // |args| are passed through the various JavaScript logging functions such as
 // console.log. Returns a string appropriate for logging with LOG(severity).
 std::string LogArgs2String(const v8::FunctionCallbackInfo<v8::Value>& args) {
   std::string message;
   bool first = true;
   for (int i = 0; i < args.Length(); i++) {
-    v8::HandleScope handle_scope;
+    v8::HandleScope handle_scope(v8::Isolate::GetCurrent());
     if (first)
       first = false;
     else
       message += " ";
 
     v8::String::Utf8Value str(args[i]);
     message += *str;
   }
   return message;
 }
@@ skipping 172 matching lines @@
     FAIL() << ExceptionToString(try_catch);
 
   v8::Handle<v8::Value> result = script->Run();
   // Ensure the script ran without errors.
   if (result.IsEmpty())
     FAIL() << ExceptionToString(try_catch);
 }
 
 std::string V8UnitTest::ExceptionToString(const v8::TryCatch& try_catch) {
   std::string str;
-  v8::HandleScope handle_scope;
+  v8::HandleScope handle_scope(v8::Isolate::GetCurrent());
   v8::String::Utf8Value exception(try_catch.Exception());
   v8::Local<v8::Message> message(try_catch.Message());
   if (message.IsEmpty()) {
     str.append(base::StringPrintf("%s\n", *exception));
   } else {
     v8::String::Utf8Value filename(message->GetScriptResourceName());
     int linenum = message->GetLineNumber();
     int colnum = message->GetStartColumn();
     str.append(base::StringPrintf(
         "%s:%i:%i %s\n", *filename, linenum, colnum, *exception));
@@ skipping 27 matching lines @@
 void V8UnitTest::Log(const v8::FunctionCallbackInfo<v8::Value>& args) {
   LOG(INFO) << LogArgs2String(args);
 }
 
 void V8UnitTest::Error(const v8::FunctionCallbackInfo<v8::Value>& args) {
   had_errors = true;
   LOG(ERROR) << LogArgs2String(args);
 }
 
 void V8UnitTest::ChromeSend(const v8::FunctionCallbackInfo<v8::Value>& args) {
-  v8::HandleScope handle_scope;
+  v8::HandleScope handle_scope(v8::Isolate::GetCurrent());
   // We expect to receive 2 args: ("testResult", [ok, message]). However,
   // chrome.send may pass only one. Therefore we need to ensure we have at least
   // 1, then ensure that the first is "testResult" before checking again for 2.
   EXPECT_LE(1, args.Length());
   if (::testing::Test::HasNonfatalFailure())
     return;
   v8::String::Utf8Value message(args[0]);
   EXPECT_EQ("testResult", std::string(*message, message.length()));
   if (::testing::Test::HasNonfatalFailure())
     return;
   EXPECT_EQ(2, args.Length());
   if (::testing::Test::HasNonfatalFailure())
     return;
   v8::Handle<v8::Array> testResult(args[1].As<v8::Array>());
   EXPECT_EQ(2U, testResult->Length());
   if (::testing::Test::HasNonfatalFailure())
     return;
   testResult_ok = testResult->Get(0)->BooleanValue();
   if (!testResult_ok) {
     v8::String::Utf8Value message(testResult->Get(1));
     LOG(ERROR) << *message;
   }
 }