Remove unsafe access hacks from ScopedPersistent.

chrome/renderer/extensions/messaging_bindings.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 "chrome/renderer/extensions/messaging_bindings.h"
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 174 matching lines @@
           base::Bind(&GCCallback::RunCallback, base::Owned(self)));
     }
 
     GCCallback(v8::Handle<v8::Object> object,
                v8::Handle<v8::Function> callback,
                v8::Isolate* isolate)
         : object_(object), callback_(callback), isolate_(isolate) {}
 
     void RunCallback() {
       v8::HandleScope handle_scope(isolate_);
-      v8::Handle<v8::Context> context = callback_->CreationContext();
+      v8::Handle<v8::Function> callback = callback_.NewHandle(isolate_);
+      v8::Handle<v8::Context> context = callback->CreationContext();
       if (context.IsEmpty())
         return;
       v8::Context::Scope context_scope(context);
       WebKit::WebScopedMicrotaskSuppression suppression;
-      callback_->Call(context->Global(), 0, NULL);
+      callback->Call(context->Global(), 0, NULL);
     }
 
     extensions::ScopedPersistent<v8::Object> object_;
     extensions::ScopedPersistent<v8::Function> callback_;
     v8::Isolate* isolate_;
 
     DISALLOW_COPY_AND_ASSIGN(GCCallback);
   };
 
   // void BindToGC(object, callback)
@@ skipping 152 matching lines @@
     } else {
       arguments.push_back(v8::Null());
     }
     (*it)->module_system()->CallModuleMethod("messaging",
                                              "dispatchOnDisconnect",
                                              &arguments);
   }
 }
 
 }  // namespace extensions