Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(407)

Unified Diff: Source/bindings/v8/custom/V8InjectedScriptManager.cpp

Issue 307943002: Oilpan: Prepare moving InspectorController and InspectorAgents to oilpan. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/bindings/v8/custom/V8InjectedScriptManager.cpp
diff --git a/Source/bindings/v8/custom/V8InjectedScriptManager.cpp b/Source/bindings/v8/custom/V8InjectedScriptManager.cpp
index 0aa3d02f51e9e681748b5f31c79e6e8442a60963..e0421d056fe061fad55987e98f40eb77853e9589 100644
--- a/Source/bindings/v8/custom/V8InjectedScriptManager.cpp
+++ b/Source/bindings/v8/custom/V8InjectedScriptManager.cpp
@@ -48,7 +48,7 @@ namespace WebCore {
struct InjectedScriptManager::CallbackData {
ScopedPersistent<v8::Object> handle;
- RefPtr<InjectedScriptHost> host;
+ RefPtrWillBePersistent<InjectedScriptHost> host;
};
static v8::Local<v8::Object> createInjectedScriptHostV8Wrapper(InjectedScriptHost* host, v8::Isolate* isolate)
@@ -63,7 +63,7 @@ static v8::Local<v8::Object> createInjectedScriptHostV8Wrapper(InjectedScriptHos
// Avoid setting the wrapper if allocation failed.
return v8::Local<v8::Object>();
}
- V8DOMWrapper::setNativeInfo(instanceTemplate, &V8InjectedScriptHost::wrapperTypeInfo, host);
+ V8DOMWrapper::setNativeInfoForHiddenWrapper(instanceTemplate, &V8InjectedScriptHost::wrapperTypeInfo, host);
haraken 2014/06/12 05:53:00 Should't this be setNativeInfoWithPersistentHandle
keishi 2014/06/13 03:37:32 Done. Wasn't sure what to use for the PersistentNo
// Create a weak reference to the v8 wrapper of InspectorBackend to deref
// InspectorBackend when the wrapper is garbage collected.
InjectedScriptManager::CallbackData* data = new InjectedScriptManager::CallbackData;
« no previous file with comments | « no previous file | Source/core/inspector/CodeGeneratorInstrumentation.py » ('j') | Source/core/inspector/InjectedScriptHost.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698