eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-04-26 01:35:08 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Avoid style computation if document is detached

Browse Source
This commit is contained in:
Tim Ledbetter
2026-03-18 22:10:52 +00:00
committed by Jelle Raaijmakers
parent 26389363ad
commit c173a66754
Notes: github-actions[bot] 2026-03-19 13:19:03 +00:00 
Author: https://github.com/tcl3
Commit: https://github.com/LadybirdBrowser/ladybird/commit/c173a667542
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/8503
Reviewed-by: https://github.com/gmta ✅
3 changed files with 22 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
Libraries/LibWeb/CSS/CSSStyleProperties.cpp
View File

@@ -540,10 +540,19 @@ Optional<StyleProperty> CSSStyleProperties::get_direct_property(PropertyNameAndI
auto abstract_element = *owner_node();
// https://www.w3.org/TR/cssom-1/#dom-window-getcomputedstyle
// NB: This is a partial enforcement of step 5 ("If elt is connected, ...")
if (!abstract_element.element().is_connected())
// https://drafts.csswg.org/cssom/#dom-window-getcomputedstyle
// NB: This is a partial enforcement of step 5:
// If [...] elt is connected, part of the flat tree, and its shadow-including root has a browsing context which
// either doesn't have a browsing context container, or whose browsing context container is being rendered.
auto& element = abstract_element.element();
if (!element.is_connected())
return {};
auto browsing_context = element.shadow_including_root().document().browsing_context();
if (!browsing_context)
return {};
// FIXME: Check if the element is part of the flat tree.
// FIXME: Check that the browsing context either doesn't have a browsing context container, or that its
// browsing context container is being rendered.
// NB: We grab the layout node before deciding whether update_layout() is needed.
// For properties that don't need layout or a layout node (the else branch below),