Bug 67273 – Remove INSTANTIATE_ALL_EVENT_INITIALIZING_CONSTRUCTORS from EventConstructors.h

• Status: RESOLVED
• Resolution: WONTFIX
• Component: DOM

[Kentaro Hara]

Using INSTANTIATE_ALL_EVENT_INITIALIZING_CONSTRUCTORS in EventConstructors.h means that the set of constructors implemented in JSC and in V8 must be synchronized. In other words, if we add a constructor to JSC, then we need to add it to V8 at the same time, which makes it difficult to add constructors to JSC and to V8 separately. 

We should remove INSTANTIATE_ALL_EVENT_INITIALIZING_CONSTRUCTORS from EventConstructors.h and describe INSTANTIATE_INITIALIZING_CONSTRUCTOR_FOR_* in JSEventConstructors.cpp and V8EventConstructors.cpp.

[Kentaro Hara]

Created attachment 105761 [details]
Patch

[Alexey Proskuryakov]

> In other words, if we add a constructor to JSC, then we need to add it to V8 at the same time, which makes it difficult to add constructors to JSC and to V8 separately. 

Keeping various WebKit based browsers compatible is a good thing. Why would it be good to add constructors separately even without a feature define?

[Kentaro Hara]

(In reply to comment #2)
> > In other words, if we add a constructor to JSC, then we need to add it to V8 at the same time, which makes it difficult to add constructors to JSC and to V8 separately. 
> 
> Keeping various WebKit based browsers compatible is a good thing. Why would it be good to add constructors separately even without a feature define?

Yeah, I do agree that we should synchronize constructors for JSC and V8. What I wanted to mean is that using that macro makes it impossible to add a constructor to JSC and to V8 _as_a_separate_patch_. In fact, recently Sam and I have been adding constructors for JSC and V8 in separate patches step by step. 

However, it may be now OK to decide that we must implement a constructor for JSC and V8 in one patch.

[Sam Weinig]

I don't think this is a good idea.  I add the ALL variant to keep the two implementations in line.

[Kentaro Hara]

(In reply to comment #4)
> I don't think this is a good idea.  I add the ALL variant to keep the two implementations in line.

OK. Let's keep ALL. However, I would like to commit patches for JSC and V8 separately in order to reduce the size of each patch. So, until all events are implemented, I will use ALL for JSC but will not use ALL for V8.