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

Source/core/css/SelectorChecker.cpp

Index: Source/core/css/SelectorChecker.cpp
diff --git a/Source/core/css/SelectorChecker.cpp b/Source/core/css/SelectorChecker.cpp
index 5f4e62c301544cd3439a02905e558445a1536ee6..23f3acaf0412256026b41d82a76278b30d4ec907 100644
--- a/Source/core/css/SelectorChecker.cpp
+++ b/Source/core/css/SelectorChecker.cpp
@@ -463,13 +463,10 @@ static bool anyAttributeMatches(Element& element, CSSSelector::Match match, cons
     // Currently all lazy properties have a null namespace, so only pass localName().
     element.synchronizeAttribute(selectorAttr.localName());
 
-    if (!element.hasAttributesWithoutUpdate())
-        return false;
-
     const AtomicString& selectorValue = selector.value();
     bool caseInsensitive = selector.attributeMatchType() == CSSSelector::CaseInsensitive;
 
-    AttributeCollection attributes = element.attributes();
+    AttributeCollection attributes = element.attributesWithoutUpdate();
     AttributeCollection::const_iterator end = attributes.end();
     for (AttributeCollection::const_iterator it = attributes.begin(); it != end; ++it) {
         const Attribute& attributeItem = *it;