LibWeb: Add Element::for_each_attribute(Function<void(Attr const&)>)

This is useful when you are wanting to interate over attributes and need
more information than just the name and value.

Userland/Libraries/LibWeb/DOM/Element.cpp

@@ -1885,12 +1885,17 @@ void Element::set_prefix(Optional<FlyString> value)
     m_qualified_name.set_prefix(move(value));
 }
 
+void Element::for_each_attribute(Function<void(Attr const&)> callback) const
+{
+    for (size_t i = 0; i < m_attributes->length(); ++i)
+        callback(*m_attributes->item(i));
+}
+
 void Element::for_each_attribute(Function<void(FlyString const&, DeprecatedString const&)> callback) const
 {
-    for (size_t i = 0; i < m_attributes->length(); ++i) {
-        auto const* attribute = m_attributes->item(i);
-        callback(attribute->name(), attribute->value().to_deprecated_string());
-    }
+    for_each_attribute([&callback](Attr const& attr) {
+        callback(attr.name(), attr.value().to_deprecated_string());
+    });
 }
 
 Layout::NodeWithStyle* Element::layout_node()

Userland/Libraries/LibWeb/DOM/Element.h

@@ -137,6 +137,8 @@ public:
     int client_width() const;
     int client_height() const;
 
+    void for_each_attribute(Function<void(Attr const&)>) const;
+
     void for_each_attribute(Function<void(FlyString const&, DeprecatedString const&)>) const;
 
     bool has_class(FlyString const&, CaseSensitivity = CaseSensitivity::CaseSensitive) const;