6 Dec
2023
6 Dec
'23
5:55 p.m.
Jacek Caban (@jacek) commented about dlls/mshtml/htmlevent.c:
static const tid_t HTMLEventObj_iface_tids[] = { IHTMLEventObj_tid,
- IHTMLEventObj2_tid,
- IHTMLEventObj3_tid,
- IHTMLEventObj4_tid,
- IHTMLEventObj5_tid,
- IHTMLEventObj6_tid,
C stubs are generally fine, but I don't think that exposing them to JS right away is a good idea. Web pages can often deal with missing properties, while providing stubs like that will just throw an error. There is no strict rule when to expose new interfaces, I usually did that together with at least a partial implementation of the interface (which also meant that I had a use case requiring that).
Anyway, a few days before the code freeze does not feel like a good moment for exposing a bunch of stubs to JS, I'd suggest to add only C stubs for now.