Move HTMLDocument::isCaseSensitiveAttribute into SelectorChecker

third_party/WebKit/Source/core/css/SelectorChecker.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 479 matching lines @@
         // It they start the same, check for exact match or following '-':
         if (value.length() != selectorValue.length() && value[selectorValue.length()] != '-')
             return false;
         return true;
     default:
         ASSERT_NOT_REACHED();
         return false;
     }
 }
 
+static HashSet<StringImpl*>* createHtmlCaseInsensitiveAttributesSet()
+{
+    // This list of attributes is from
+    // https://html.spec.whatwg.org/#case-sensitivity-of-selectors
+    HashSet<StringImpl*>* attrSet = new HashSet<StringImpl*>;
+
+    const QualifiedName* caseInsensitiveAttributes[] = {
+        &accept_charsetAttr, &acceptAttr, &alignAttr, &alinkAttr, &axisAttr,
+        &bgcolorAttr,
+        &charsetAttr, &checkedAttr, &clearAttr, &codetypeAttr, &colorAttr, &compactAttr,
+        &declareAttr, &deferAttr, &dirAttr, &directionAttr, &disabledAttr,
+        &enctypeAttr,
+        &faceAttr, &frameAttr,
+        &hreflangAttr, &http_equivAttr,
+        &langAttr, &languageAttr, &linkAttr,
+        &mediaAttr, &methodAttr, &multipleAttr,
+        &nohrefAttr, &noresizeAttr, &noshadeAttr, &nowrapAttr,
+        &readonlyAttr, &relAttr, &revAttr, &rulesAttr,
+        &scopeAttr, &scrollingAttr, &selectedAttr, &shapeAttr,
+        &targetAttr, &textAttr, &typeAttr,
+        &valignAttr, &valuetypeAttr, &vlinkAttr };
+
+    attrSet->reserveCapacityForSize(WTF_ARRAY_LENGTH(caseInsensitiveAttributes));
+    for (const QualifiedName* attr : caseInsensitiveAttributes)
+        attrSet->add(attr->localName().impl());
+
+    return attrSet;
+}
+
+static bool isHtmlCaseSensitiveAttribute(const QualifiedName& attributeName)
+{
+    static HashSet<StringImpl*>* htmlCaseInsensitiveAttributesSet = createHtmlCaseInsensitiveAttributesSet();
+    bool isPossibleHTMLAttr = !attributeName.hasPrefix() && (attributeName.namespaceURI() == nullAtom);
+    return !isPossibleHTMLAttr || !htmlCaseInsensitiveAttributesSet->contains(attributeName.localName().impl());
+}
+
 static bool anyAttributeMatches(Element& element, CSSSelector::MatchType match, const CSSSelector& selector)
 {
     const QualifiedName& selectorAttr = selector.attribute();
     ASSERT(selectorAttr.localName() != starAtom); // Should not be possible from the CSS grammar.
 
     // Synchronize the attribute in case it is lazy-computed.
     // Currently all lazy properties have a null namespace, so only pass localName().
     element.synchronizeAttribute(selectorAttr.localName());
 
     const AtomicString& selectorValue = selector.value();
     TextCaseSensitivity caseSensitivity = (selector.attributeMatch() == CSSSelector::CaseInsensitive) ? TextCaseASCIIInsensitive : TextCaseSensitive;
 
     AttributeCollection attributes = element.attributesWithoutUpdate();
     for (const auto& attributeItem: attributes) {
         if (!attributeItem.matches(selectorAttr))
             continue;
 
         if (attributeValueMatches(attributeItem, match, selectorValue, caseSensitivity))
             return true;
 
         if (caseSensitivity == TextCaseASCIIInsensitive) {
             if (selectorAttr.namespaceURI() != starAtom)
                 return false;
             continue;
         }
 
-        // Legacy dictates that values of some attributes should be compared in
-        // a case-insensitive manner regardless of whether the case insensitive
-        // flag is set or not.
-        bool legacyCaseInsensitive = element.document().isHTMLDocument() && !HTMLDocument::isCaseSensitiveAttribute(selectorAttr);
+        // https://html.spec.whatwg.org/#case-sensitivity-of-selectors
+        // TODO(foolip): The spec says "HTML element in an HTML document", but
+        // this applies to any element in an HTML document.
+        bool legacyCaseInsensitive = element.document().isHTMLDocument() && !isHtmlCaseSensitiveAttribute(selectorAttr);
 
         // If case-insensitive, re-check, and count if result differs.
         // See http://code.google.com/p/chromium/issues/detail?id=327060
         if (legacyCaseInsensitive && attributeValueMatches(attributeItem, match, selectorValue, TextCaseASCIIInsensitive)) {
             UseCounter::count(element.document(), UseCounter::CaseInsensitiveAttrSelectorMatch);
             return true;
         }
         if (selectorAttr.namespaceURI() != starAtom)
             return false;
     }
@@ skipping 622 matching lines @@
 }
 
 bool SelectorChecker::matchesFocusPseudoClass(const Element& element)
 {
     if (InspectorInstrumentation::forcePseudoState(const_cast<Element*>(&element), CSSSelector::PseudoFocus))
         return true;
     return element.focused() && isFrameFocused(element);
 }
 
 } // namespace blink