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

Source/core/events/MessageEvent.cpp

Index: Source/core/events/MessageEvent.cpp
diff --git a/Source/core/events/MessageEvent.cpp b/Source/core/events/MessageEvent.cpp
index 67465a1981fb995fef00b09090cf8fd3a33a43cf..e4c996f5a4de401e6661c8ab090bd1ca586a0255 100644
--- a/Source/core/events/MessageEvent.cpp
+++ b/Source/core/events/MessageEvent.cpp
@@ -198,7 +198,17 @@ void MessageEvent::trace(Visitor* visitor)
 v8::Handle<v8::Object> MessageEvent::wrap(v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     v8::Handle<v8::Object> wrapper = Event::wrap(creationContext, isolate);
+    return associateWithWrapperInternal(wrapper, isolate);
+}
 
+v8::Handle<v8::Object> MessageEvent::associateWithWrapper(const WrapperTypeInfo* wrapperType, v8::Handle<v8::Object> wrapper, v8::Isolate* isolate)
+{
+    Event::associateWithWrapper(wrapperType, wrapper, isolate);
+    return associateWithWrapperInternal(wrapper, isolate);

[haraken, 2014/09/05 16:41:39]

I guess that associateWithWrapperInternal should be executed in wrap, but should
not be executed in associateWithWrapper.

[Yuki, 2014/09/08 07:56:05]

We need associateWithWrapperInternal for callback constructor's case, too. 
That's why MessageEvent needed Custom=Wrap.

+}
+
+v8::Handle<v8::Object> MessageEvent::associateWithWrapperInternal(v8::Handle<v8::Object> wrapper, v8::Isolate* isolate)
+{
     // Ensures a wrapper is created for the data to return now so that V8 knows how
     // much memory is used via the wrapper. To keep the wrapper alive, it's set to
     // the wrapper of the MessageEvent as a hidden value.