summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibWeb/DOM
diff options
context:
space:
mode:
authorTimothy Flynn <trflynn89@pm.me>2022-03-23 18:55:54 -0400
committerAndreas Kling <kling@serenityos.org>2022-03-24 03:35:11 +0100
commit5608bc4eaf6bbc72def4cdb09052a6e91e14a3c1 (patch)
tree4bc6851c520c52647b4f88d9db17d8281b3a11dd /Userland/Libraries/LibWeb/DOM
parentf7f0195fae3f4a1296213256b5bda12b3264c5dc (diff)
downloadserenity-5608bc4eaf6bbc72def4cdb09052a6e91e14a3c1.zip
LibWeb: Remove inheritance of FormAssociatedElement from HTMLElement
HTMLObjectElement will need to be both a FormAssociatedElement and a BrowsingContextContainer. Currently, both of these classes inherit from HTMLElement. This can work in C++, but is generally frowned upon, and doesn't play particularly well with the rest of LibWeb. Instead, we can essentially revert commit 3bb5c62 to remove HTMLElement from FormAssociatedElement's hierarchy. This means that objects such as HTMLObjectElement individually inherit from FormAssociatedElement and HTMLElement now. Some caveats are: * FormAssociatedElement still needs to know when the HTMLElement is inserted into and removed from the DOM. This hook is automatically injected via a macro now, while still allowing classes like HTMLInputElement to also know when the element is inserted. * Casting from a DOM::Element to a FormAssociatedElement is now a sideways cast, rather than directly following an inheritance chain. This means static_cast cannot be used here; but we can safely use dynamic_cast since the only 2 instances of this already use RTTI to verify the cast.
Diffstat (limited to 'Userland/Libraries/LibWeb/DOM')
-rw-r--r--Userland/Libraries/LibWeb/DOM/EventTarget.cpp8
1 files changed, 5 insertions, 3 deletions
diff --git a/Userland/Libraries/LibWeb/DOM/EventTarget.cpp b/Userland/Libraries/LibWeb/DOM/EventTarget.cpp
index 2a2e113a03..95c74d0090 100644
--- a/Userland/Libraries/LibWeb/DOM/EventTarget.cpp
+++ b/Userland/Libraries/LibWeb/DOM/EventTarget.cpp
@@ -336,9 +336,11 @@ Bindings::CallbackType* EventTarget::get_current_value_of_event_handler(FlyStrin
// 5. If element is not null and element has a form owner, let form owner be that form owner. Otherwise, let form owner be null.
RefPtr<HTML::HTMLFormElement> form_owner;
if (is<HTML::FormAssociatedElement>(element.ptr())) {
- auto& form_associated_element = verify_cast<HTML::FormAssociatedElement>(*element);
- if (form_associated_element.form())
- form_owner = form_associated_element.form();
+ auto* form_associated_element = dynamic_cast<HTML::FormAssociatedElement*>(element.ptr());
+ VERIFY(form_associated_element);
+
+ if (form_associated_element->form())
+ form_owner = form_associated_element->form();
}
// 6. Let settings object be the relevant settings object of document.