Jacek Caban (@jacek) commented about dlls/mshtml/mshtml_private.h:
};
typedef struct {
- DispatchEx dispex;
- HTMLDOMNode node;
It's impossible to review that part properly without more context. For example, I could imagine implementing IE9+ attributes as a completely separate object rather than trying to fit everything into the existing one, but I can't say without seeing more context.
In this MR, you're only using it to store the owner document, but you could just as easily store that in the attribute itself or leave the `ownerDocument` patch out of this series altogether. If it turns out to be necessary, we can move it in a later MR that actually makes use of it, when it will be reviewable.