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

Source/core/inspector/InjectedScriptManager.h

 /*
  * Copyright (C) 2007 Apple Inc. All rights reserved.
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 31 matching lines @@
 class InjectedScript;
 class InjectedScriptHost;
 class ScriptValue;
 
 class InjectedScriptManager : public NoBaseWillBeGarbageCollectedFinalized<InjectedScriptManager> {
     WTF_MAKE_NONCOPYABLE(InjectedScriptManager);
     WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED;
 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();
     static PassOwnPtrWillBeRawPtr<InjectedScriptManager> createForWorker();
     ~InjectedScriptManager();
     void trace(Visitor*);
 
     void disconnect();
 
     InjectedScriptHost* injectedScriptHost();
@@ skipping 28 matching lines @@
     RefPtrWillBeMember<InjectedScriptHost> m_injectedScriptHost;
     InspectedStateAccessCheck m_inspectedStateAccessCheck;
     typedef HashMap<RefPtr<ScriptState>, int> ScriptStateToId;
     ScriptStateToId m_scriptStateToId;
     HashSet<OwnPtr<CallbackData> > m_callbackDataSet;
 };
 
 } // namespace blink
 
 #endif // !defined(InjectedScriptManager_h)