Remove calls to HandleScope default ctor.

net/proxy/proxy_resolver_v8.cc

 // Copyright (c) 2012 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 "net/proxy/proxy_resolver_v8.h"
 
 #include <algorithm>
 #include <cstdio>
 
 #include "base/basictypes.h"
@@ skipping 166 matching lines @@
   DCHECK(IsStringASCII(ascii));
   size_t length = strlen(ascii);
   if (length <= kMaxStringBytesForCopy)
     return v8::String::New(ascii, length);
   return v8::String::NewExternal(new V8ExternalASCIILiteral(ascii, length));
 }
 
 // Stringizes a V8 object by calling its toString() method. Returns true
 // on success. This may fail if the toString() throws an exception.
 bool V8ObjectToUTF16String(v8::Handle<v8::Value> object,
-                           base::string16* utf16_result) {
+                           base::string16* utf16_result,
+                           v8::Isolate* isolate) {
   if (object.IsEmpty())
     return false;
 
-  v8::HandleScope scope;
+  v8::HandleScope scope(isolate);
   v8::Local<v8::String> str_object = object->ToString();
   if (str_object.IsEmpty())
     return false;
   *utf16_result = V8StringToUTF16(str_object);
   return true;
 }
 
 // Extracts an hostname argument from |args|. On success returns true
 // and fills |*hostname| with the result.
 bool GetHostnameArgument(const v8::FunctionCallbackInfo<v8::Value>& args,
@@ skipping 298 matching lines @@
     return (*function)->IsFunction();
   }
 
   // Handle an exception thrown by V8.
   void HandleError(v8::Handle<v8::Message> message) {
     base::string16 error_message;
     int line_number = -1;
 
     if (!message.IsEmpty()) {
       line_number = message->GetLineNumber();
-      V8ObjectToUTF16String(message->Get(), &error_message);
+      V8ObjectToUTF16String(message->Get(), &error_message, isolate_);
     }
 
     js_bindings()->OnError(line_number, error_message);
   }
 
   // Compiles and runs |script| in the current V8 context.
   // Returns OK on success, otherwise an error code.
   int RunScript(v8::Handle<v8::String> script, const char* script_name) {
     v8::TryCatch try_catch;
 
@@ skipping 19 matching lines @@
   static void AlertCallback(const v8::FunctionCallbackInfo<v8::Value>& args) {
     Context* context =
         static_cast<Context*>(v8::External::Cast(*args.Data())->Value());
 
     // Like firefox we assume "undefined" if no argument was specified, and
     // disregard any arguments beyond the first.
     base::string16 message;
     if (args.Length() == 0) {
       message = ASCIIToUTF16("undefined");
     } else {
-      if (!V8ObjectToUTF16String(args[0], &message))
+      if (!V8ObjectToUTF16String(args[0], &message, args.GetIsolate()))
         return;  // toString() threw an exception.
     }
 
     context->js_bindings()->Alert(message);
   }
 
   // V8 callback for when "myIpAddress()" is invoked by the PAC script.
   static void MyIpAddressCallback(
       const v8::FunctionCallbackInfo<v8::Value>& args) {
     DnsResolveCallbackHelper(args, JSBindings::MY_IP_ADDRESS);
@@ skipping 238 matching lines @@
   if (!g_default_isolate_)
     return 0;
 
   v8::Locker locked(g_default_isolate_);
   v8::HeapStatistics heap_statistics;
   g_default_isolate_->GetHeapStatistics(&heap_statistics);
   return heap_statistics.used_heap_size();
 }
 
 }  // namespace net