Chromium Code Reviews| 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; |