[oilpan]: optimize the way we allocate persistent handles in wrappers.

Source/core/inspector/InjectedScriptManager.h

Index: Source/core/inspector/InjectedScriptManager.h
diff --git a/Source/core/inspector/InjectedScriptManager.h b/Source/core/inspector/InjectedScriptManager.h
index 289613a5dbfdb75e063fa28b9780d85da15a3a31..f35fb937731b805206562b1cb832e5d8fad30efd 100644
--- a/Source/core/inspector/InjectedScriptManager.h
+++ b/Source/core/inspector/InjectedScriptManager.h
@@ -49,8 +49,17 @@ class InjectedScriptManager : public NoBaseWillBeGarbageCollectedFinalized<Injec
 public:
     struct CallbackData {
         ScopedPersistent<v8::Object> handle;
-        RefPtrWillBePersistent<InjectedScriptHost> host;
+#if ENABLE(OILPAN)
+        WrapperPersistent<InjectedScriptHost>* hostPtr;

[zerny-chromium, 2014/09/01 14:13:37]

Why the name change?

[haraken, 2014/09/02 05:22:05]

InjectedScriptManager is a very special case, so you don't need to change this
in this CL. I'm planning to replace the CallbackData with
DOMWrapperWorld::DOMObjectHolder in the near future.

[wibling-chromium, 2014/09/02 11:19:37]

If i don't change it I cannot change the setXXX methods to consistently use a
WrapperPersistentNode since this will still use a normal Persistent.

[wibling-chromium, 2014/09/02 11:19:37]

Since it it now a pointer to the WrapperPersistent rather than an embedded
RefPtr.

+#else
+        RefPtr<InjectedScriptHost> host;
+#endif
         InjectedScriptManager* injectedScriptManager;
+
+#if ENABLE(OILPAN)
+        CallbackData() : hostPtr(0) { }
+        ~CallbackData();
+#endif
     };
 
     static PassOwnPtrWillBeRawPtr<InjectedScriptManager> createForPage();