On Wed May 14 07:59:14 2025 +0000, Jacek Caban wrote:
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.
Separate object might work, but needs a deeper look and will need some reworks, not entirely sure yet if it's a simplification to what I have, but honestly both require special casing anyway, so maybe it is.
Anyway, I made it hold document now without going through the node change.