On Fri Aug 18 13:29:43 2023 +0000, Jacek Caban wrote:
Maybe, but we don't have ref value getter currently. I guess you we could hardcode `nsrefcnt` layout until the next Gecko release, I'm not sure. Let's take care of that in another MR. It would be good to move that commit closer to the end of conversion series so that we can see if there are more cases like that. Other patches look good to me.
Oh yeah, I forgot it's a ccref… I put it in this MR because I wanted to get rid of `EventTarget_QI_no_cc`. I think the "ugly" solution isn't so bad after all?
I can't really see a different solution without requiring wine-gecko changes or even more ugly stuff (hardcoding ref layout), and at that point we could just switch over from it, IMO.
Well I'll keep it for now but if you still find it too ugly I'll just remove the patch.