From 609a89cfb0819126c61d49e0b3bf3be38b833c35 Mon Sep 17 00:00:00 2001 From: Sam Atkins Date: Fri, 27 Jan 2023 13:13:25 +0000 Subject: [PATCH] LibWeb: Correct logic in TreeWalker::next_node() We were creating a new `result` variable here, when we should have been assigning to the original one. --- Userland/Libraries/LibWeb/DOM/TreeWalker.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibWeb/DOM/TreeWalker.cpp b/Userland/Libraries/LibWeb/DOM/TreeWalker.cpp index 419cd7fd80..5f65d92af8 100644 --- a/Userland/Libraries/LibWeb/DOM/TreeWalker.cpp +++ b/Userland/Libraries/LibWeb/DOM/TreeWalker.cpp @@ -187,7 +187,7 @@ JS::ThrowCompletionOr> TreeWalker::next_node() node = *node->first_child(); // 2. Set result to the result of filtering node within this. - auto result = TRY(filter(*node)); + result = TRY(filter(*node)); // 3. If result is FILTER_ACCEPT, then set this’s current to node and return node. if (result == NodeFilter::FILTER_ACCEPT) {