Make Element::attributes() less error-prone and simplify call sites

Source/core/editing/MarkupAccumulator.cpp

Index: Source/core/editing/MarkupAccumulator.cpp
diff --git a/Source/core/editing/MarkupAccumulator.cpp b/Source/core/editing/MarkupAccumulator.cpp
index e67e8380a13757a989abfff4a0c470f4abefffd8..9fd11109fff69b932b390c7646974ae01fa6e056 100644
--- a/Source/core/editing/MarkupAccumulator.cpp
+++ b/Source/core/editing/MarkupAccumulator.cpp
@@ -385,12 +385,10 @@ void MarkupAccumulator::appendElement(StringBuilder& result, Element& element, N
 {
     appendOpenTag(result, element, namespaces);
 
-    if (element.hasAttributes()) {
-        AttributeCollection attributes = element.attributes();
-        AttributeCollection::const_iterator end = attributes.end();
-        for (AttributeCollection::const_iterator it = attributes.begin(); it != end; ++it)
-            appendAttribute(result, element, *it, namespaces);
-    }
+    AttributeCollection attributes = element.attributes();
+    AttributeCollection::const_iterator end = attributes.end();
+    for (AttributeCollection::const_iterator it = attributes.begin(); it != end; ++it)
+        appendAttribute(result, element, *it, namespaces);
 
     // Give an opportunity to subclasses to add their own attributes.
     appendCustomAttributes(result, element, namespaces);