Reduce the number of deprecated v8 API entry calls. Disable V8 deprecations for proxy resolver.

To do this, we can simply use v8::Arguments::GetIsolate in callbacks,
which is even more efficient than before.

Disabling V8 deprecations for proxy_resolver_v8.cc is only a temporary
measure until the remaining 4 calls to v8::Locker are fixed.

BUG=v8:2487


Review URL: https://chromiumcodereview.appspot.com/12045044

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178588 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 4 deletions

diff --git a/net/proxy/proxy_resolver_v8.cc b/net/proxy/proxy_resolver_v8.cc
index 5150317..6956dae 100644
--- a/net/proxy/proxy_resolver_v8.cc
+++ b/net/proxy/proxy_resolver_v8.cc
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#define V8_DISABLE_DEPRECATIONS 1
+
 #include "net/proxy/proxy_resolver_v8.h"
 
 #include <algorithm>
@@ -595,7 +597,7 @@ class ProxyResolverV8::Context {
     bool success;
 
     {
-      v8::Unlocker unlocker;
+      v8::Unlocker unlocker(args.GetIsolate());
       ScopedHostResolve scoped_host_resolve(context);
 
       // We shouldn't be called with any arguments, but will not complain if
@@ -618,7 +620,7 @@ class ProxyResolverV8::Context {
     bool success;
 
     {
-      v8::Unlocker unlocker;
+      v8::Unlocker unlocker(args.GetIsolate());
       ScopedHostResolve scoped_host_resolve(context);
 
       // We shouldn't be called with any arguments, but will not complain if
@@ -645,7 +647,7 @@ class ProxyResolverV8::Context {
     bool success;
 
     {
-      v8::Unlocker unlocker;
+      v8::Unlocker unlocker(args.GetIsolate());
       ScopedHostResolve scoped_host_resolve(context);
       success = context->js_bindings_->DnsResolve(hostname, &ip_address);
     }
@@ -667,7 +669,7 @@ class ProxyResolverV8::Context {
     bool success;
 
     {
-      v8::Unlocker unlocker;
+      v8::Unlocker unlocker(args.GetIsolate());
       ScopedHostResolve scoped_host_resolve(context);
       success = context->js_bindings_->DnsResolveEx(hostname, &ip_address_list);
     }