14 Nov
2022
14 Nov
'22
6:20 p.m.
On Mon Nov 14 18:20:01 2022 +0000, Gabriel Ivăncescu wrote:
It was just for consistency with the other places where it's used. Should I drop the check? (I guess, in theory, it could happen that new node fails to allocate while keeping old IEventTarget around, but probably not worth in practice) BTW I noticed something else, I probably should move the document_mode and window retrieval to navigate() to keep the globals synced as it's expected (in the tests). As I mentioned in !1125, those checks are leftovers. I think it's better to drop them now.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1317#note_15733