From 192f4b0258b04197a3aba5eaa2cbedb34dce672c Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sun, 27 Mar 2022 20:12:45 +0100 Subject: [PATCH] LibWeb: Ensure lazy WindowObject creation when activating event handler --- Userland/Libraries/LibWeb/DOM/EventTarget.cpp | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/Userland/Libraries/LibWeb/DOM/EventTarget.cpp b/Userland/Libraries/LibWeb/DOM/EventTarget.cpp index 95c74d0090..ae84bd87bb 100644 --- a/Userland/Libraries/LibWeb/DOM/EventTarget.cpp +++ b/Userland/Libraries/LibWeb/DOM/EventTarget.cpp @@ -519,15 +519,18 @@ void EventTarget::activate_event_handler(FlyString const& name, HTML::EventHandl // returning the element's document's global object, which is the HTML::Window object. // For any other HTMLElement who just had an element attribute set, `this` will be that HTMLElement, so the global object is this's document's realm's global object. // For anything else, it came from JavaScript, so use the global object of the provided callback function. - // Sadly, this doesn't work if an element attribute is set on a element before any script is run, as Window::wrapper() will be null. JS::GlobalObject* global_object = nullptr; if (is_attribute == IsAttribute::Yes) { if (is(this)) { - auto* window_global_object = verify_cast(this)->wrapper(); - global_object = static_cast(window_global_object); + auto& window = verify_cast(*this); + // If an element attribute is set on a element before any script is run, Window::wrapper() will be null. + // Force creation of the global object via the Document::interpreter() lazy initialization mechanism. + if (window.wrapper() == nullptr) + window.associated_document().interpreter(); + global_object = static_cast(window.wrapper()); } else { - auto* html_element = verify_cast(this); - global_object = &html_element->document().realm().global_object(); + auto& html_element = verify_cast(*this); + global_object = &html_element.document().realm().global_object(); } } else { global_object = &event_handler.value.get().callback.cell()->global_object();