Oilpan: Move [WillBeGarbageCollected] from Node to EventTarget.

Source/bindings/tests/results/V8TestInterfaceEventTarget.h

Index: Source/bindings/tests/results/V8TestInterfaceEventTarget.h
diff --git a/Source/bindings/tests/results/V8TestInterfaceEventTarget.h b/Source/bindings/tests/results/V8TestInterfaceEventTarget.h
index c60e751e4ecee698093e929deb0285890d438217..821ee47c99b587432db4fc304d36fad29c5e7bfd 100644
--- a/Source/bindings/tests/results/V8TestInterfaceEventTarget.h
+++ b/Source/bindings/tests/results/V8TestInterfaceEventTarget.h
@@ -36,7 +36,8 @@ public:
     static void derefObject(void*);
     static EventTarget* toEventTarget(v8::Handle<v8::Object>);
     static const int eventListenerCacheIndex = v8DefaultWrapperInternalFieldCount + 0;
-    static const int internalFieldCount = v8DefaultWrapperInternalFieldCount + 1;
+    static const int persistentHandleIndex = v8DefaultWrapperInternalFieldCount + 1;
+    static const int internalFieldCount = v8DefaultWrapperInternalFieldCount + 2;

[haraken, 2014/06/02 12:41:18]

oh, this might be problematic, because this increases the size of each
EventTarget wrapper (including all Nodes) by 8 byte.

Given that persistentHandleIndex is not used in non-oilpan builds, we should
probably do something like:

#if ENABLE(OILPAN)
static const int internalFieldCount = v8DefaultWrapperInternalFieldCount + 2
#else
static const int internalFieldCount = v8DefaultWrapperInternalFieldCount + 1
#endif

(I can work on it tomorrow morning if you're not willing to touch the IDL
compiler.)

[zerny-chromium, 2014/06/02 13:06:49]

Yes, please do. I agree that we should not increase the wrapper size for
non-oilpan builds, so I'll wait on your CL before this can land.

     static inline void* toInternalPointer(TestInterfaceEventTarget* impl)
     {
         return V8EventTarget::toInternalPointer(impl);
@@ -51,7 +52,7 @@ public:
 
 private:
     friend v8::Handle<v8::Object> wrap(TestInterfaceEventTarget*, v8::Handle<v8::Object> creationContext, v8::Isolate*);
-    static v8::Handle<v8::Object> createWrapper(PassRefPtr<TestInterfaceEventTarget>, v8::Handle<v8::Object> creationContext, v8::Isolate*);
+    static v8::Handle<v8::Object> createWrapper(PassRefPtrWillBeRawPtr<TestInterfaceEventTarget>, v8::Handle<v8::Object> creationContext, v8::Isolate*);
 };
 
 inline v8::Handle<v8::Object> wrap(TestInterfaceEventTarget* impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
@@ -111,25 +112,25 @@ inline void v8SetReturnValueFast(const CallbackInfo& callbackInfo, TestInterface
     v8SetReturnValue(callbackInfo, wrapper);
 }
 
-inline v8::Handle<v8::Value> toV8(PassRefPtr<TestInterfaceEventTarget> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
+inline v8::Handle<v8::Value> toV8(PassRefPtrWillBeRawPtr<TestInterfaceEventTarget> impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     return toV8(impl.get(), creationContext, isolate);
 }
 
 template<class CallbackInfo>
-inline void v8SetReturnValue(const CallbackInfo& callbackInfo, PassRefPtr<TestInterfaceEventTarget> impl)
+inline void v8SetReturnValue(const CallbackInfo& callbackInfo, PassRefPtrWillBeRawPtr<TestInterfaceEventTarget> impl)
 {
     v8SetReturnValue(callbackInfo, impl.get());
 }
 
 template<class CallbackInfo>
-inline void v8SetReturnValueForMainWorld(const CallbackInfo& callbackInfo, PassRefPtr<TestInterfaceEventTarget> impl)
+inline void v8SetReturnValueForMainWorld(const CallbackInfo& callbackInfo, PassRefPtrWillBeRawPtr<TestInterfaceEventTarget> impl)
 {
     v8SetReturnValueForMainWorld(callbackInfo, impl.get());
 }
 
 template<class CallbackInfo, class Wrappable>
-inline void v8SetReturnValueFast(const CallbackInfo& callbackInfo, PassRefPtr<TestInterfaceEventTarget> impl, Wrappable* wrappable)
+inline void v8SetReturnValueFast(const CallbackInfo& callbackInfo, PassRefPtrWillBeRawPtr<TestInterfaceEventTarget> impl, Wrappable* wrappable)
 {
     v8SetReturnValueFast(callbackInfo, impl.get(), wrappable);
 }