LibWeb: Don't force layout when scrolling to non-existent anchor

If we're asked to scroll to an anchor element that's not actually in the
document, we don't need to perform a layout.

1 files changed, 9 insertions, 5 deletions

diff --git a/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp b/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp
index 18cbdbdcfd..e90f88cae1 100644
--- a/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp
+++ b/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp
@@ -392,12 +392,13 @@ void BrowsingContext::scroll_to(Gfx::IntPoint const& position)
 
 void BrowsingContext::scroll_to_anchor(String const& fragment)
 {
-    if (!active_document())
+    JS::GCPtr<DOM::Document> document = active_document();
+    if (!document)
         return;
 
-    auto element = active_document()->get_element_by_id(fragment);
+    auto element = document->get_element_by_id(fragment);
     if (!element) {
-        auto candidates = active_document()->get_elements_by_name(fragment);
+        auto candidates = document->get_elements_by_name(fragment);
         for (auto& candidate : candidates->collect_matching_elements()) {
             if (is<HTML::HTMLAnchorElement>(*candidate)) {
                 element = &verify_cast<HTML::HTMLAnchorElement>(*candidate);
@@ -406,9 +407,12 @@ void BrowsingContext::scroll_to_anchor(String const& fragment)
         }
     }
 
-    active_document()->force_layout();
+    if (!element)
+        return;
+
+    document->force_layout();
 
-    if (!element || !element->layout_node())
+    if (!element->layout_node())
         return;
 
     auto& layout_node = *element->layout_node();