LibWeb: Use RefPtrs more in getElementById() and getElementsByName()

Passing around Vector<Element*> is not a great idea long-term.

1 files changed, 3 insertions, 3 deletions

diff --git a/Libraries/LibWeb/DOM/Document.cpp b/Libraries/LibWeb/DOM/Document.cpp
index 97aa172646..efb64b68e5 100644
--- a/Libraries/LibWeb/DOM/Document.cpp
+++ b/Libraries/LibWeb/DOM/Document.cpp
@@ -359,12 +359,12 @@ void Document::set_hovered_node(Node* node)
     invalidate_style();
 }
 
-Vector<const Element*> Document::get_elements_by_name(const String& name) const
+NonnullRefPtrVector<Element> Document::get_elements_by_name(const String& name) const
 {
-    Vector<const Element*> elements;
+    NonnullRefPtrVector<Element> elements;
     for_each_in_subtree_of_type<Element>([&](auto& element) {
         if (element.attribute(HTML::AttributeNames::name) == name)
-            elements.append(&element);
+            elements.append(element);
         return IterationDecision::Continue;
     });
     return elements;