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

Source/bindings/core/v8/ScriptWrappable.h

Index: Source/bindings/core/v8/ScriptWrappable.h
diff --git a/Source/bindings/core/v8/ScriptWrappable.h b/Source/bindings/core/v8/ScriptWrappable.h
index 2781a976dfdbb6e16f00c1e30e6fa454bc4a4f46..f24e4edc4197ecd9203d1aaf1c51d0e3e2bd9335 100644
--- a/Source/bindings/core/v8/ScriptWrappable.h
+++ b/Source/bindings/core/v8/ScriptWrappable.h
@@ -120,6 +120,9 @@ public:
     // Creates and returns a new wrapper object.
     virtual v8::Handle<v8::Object> wrap(v8::Handle<v8::Object> creationContext, v8::Isolate*);
 
+    // Associates the instance with the existing wrapper. Returns |wrapper|.

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

Why does this need to return a wrapper? The returned wrapper should be the same
as the wrapper passed to the method.

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

Not necessary, it's just convenient that we can
    return associateWithWrapper(...);
instead of
    associateWithWrapper(...);
    return wrapper;

If you don't like this style, I'll change the return type to void.

[haraken, 2014/09/08 09:07:22]

Either is fine with me :)

+    virtual v8::Handle<v8::Object> associateWithWrapper(const WrapperTypeInfo*, v8::Handle<v8::Object> wrapper, v8::Isolate*);
+
     void setWrapper(v8::Handle<v8::Object> wrapper, v8::Isolate* isolate, const WrapperTypeInfo* wrapperTypeInfo)
     {
         ASSERT(!containsWrapper());