bindings: Introduces ScriptWrappable::associateWithWrapper in addition to wrap.

Source/core/dom/Document.cpp

Index: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index cad741aa557bb13f073105950dc6d1f627124519..788909a4c864ff6a5340e4063bcbb9db0bdcc61b 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -5772,13 +5772,16 @@ v8::Handle<v8::Object> Document::wrap(v8::Handle<v8::Object> creationContext, v8
         return wrapper;
 
     wrapperType->installConditionallyEnabledProperties(wrapper, isolate);

[haraken, 2014/09/08 08:48:45]

Probably this needs to be in associateWithWrapper, doesn't it?

[Yuki, 2014/09/08 09:00:22]

Reading bindings/templates/methods.cpp, the original code doesn't call it. 
Though, I'm not 100% sure...

+    return associateWithWrapper(wrapperType, wrapper, isolate);
+}
+
+v8::Handle<v8::Object> Document::associateWithWrapper(const WrapperTypeInfo* wrapperType, v8::Handle<v8::Object> wrapper, v8::Isolate* isolate)
+{
     wrapperType->refObject(toScriptWrappableBase());
     V8DOMWrapper::associateObjectWithWrapperNonTemplate(this, wrapperType, wrapper, isolate);
-
     DOMWrapperWorld& world = DOMWrapperWorld::current(isolate);
     if (world.isMainWorld() && frame())
         frame()->script().windowProxy(world)->updateDocumentWrapper(wrapper);
-
     return wrapper;
 }