RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-06-01 10:08:10 +00:00
Code Issues Activity Actions

LibWeb: Remove duplicate type checks in SelectorEngine

Browse source 
After confirming the rule types, we can use static_cast instead of
verify_cast to avoid some extra work.
This commit is contained in:
Andreas Kling 2022-01-19 09:47:04 +01:00
parent 77650fe886
commit ae301d2b02
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibWeb/CSS/SelectorEngine.cpp
View file

@ -203,7 +203,7 @@ static bool matches(CSS::Selector const& selector, int component_list_index, DOM
for (auto* ancestor = element.parent(); ancestor; ancestor = ancestor->parent()) {
if (!is<DOM::Element>(*ancestor))
continue;
if (matches(selector, component_list_index - 1, verify_cast<DOM::Element>(*ancestor)))
if (matches(selector, component_list_index - 1, static_cast<DOM::Element const&>(*ancestor)))
return true;
}
return false;
@ -211,7 +211,7 @@ static bool matches(CSS::Selector const& selector, int component_list_index, DOM
VERIFY(component_list_index != 0);
if (!element.parent() || !is<DOM::Element>(*element.parent()))
return false;
return matches(selector, component_list_index - 1, verify_cast<DOM::Element>(*element.parent()));
return matches(selector, component_list_index - 1, static_cast<DOM::Element const&>(*element.parent()));
case CSS::Selector::Combinator::NextSibling:
VERIFY(component_list_index != 0);
if (auto* sibling = element.previous_element_sibling())