Use ASCII case-insensitive matching for ident-likes in the CSS parser

third_party/WebKit/Source/core/css/parser/MediaQueryParser.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/css/parser/MediaQueryParser.h"
 
 #include "core/MediaTypeNames.h"
 #include "core/css/parser/CSSPropertyParser.h"
 #include "core/css/parser/CSSTokenizer.h"
 #include "wtf/Vector.h"
@@ skipping 47 matching lines @@
 }
 
 // State machine member functions start here
 void MediaQueryParser::readRestrictor(CSSParserTokenType type, const CSSParserToken& token)
 {
     readMediaType(type, token);
 }
 
 void MediaQueryParser::readMediaNot(CSSParserTokenType type, const CSSParserToken& token)
 {
-    if (type == IdentToken && equalIgnoringCase(token.value(), "not"))
+    if (type == IdentToken && token.valueEqualsIgnoringASCIICase("not"))
         setStateAndRestrict(ReadFeatureStart, MediaQuery::Not);
     else
         readFeatureStart(type, token);
 }
 
-static bool isRestrictorOrLogicalOperator(const String& tokenValue)
+static bool isRestrictorOrLogicalOperator(const CSSParserToken& token)
 {
     // FIXME: it would be more efficient to use lower-case always for tokenValue.
-    return equalIgnoringCase(tokenValue, "not")
-        || equalIgnoringCase(tokenValue, "and")
-        || equalIgnoringCase(tokenValue, "or")
-        || equalIgnoringCase(tokenValue, "only");
+    return token.valueEqualsIgnoringASCIICase("not")
+        || token.valueEqualsIgnoringASCIICase("and")
+        || token.valueEqualsIgnoringASCIICase("or")
+        || token.valueEqualsIgnoringASCIICase("only");
 }
 
 void MediaQueryParser::readMediaType(CSSParserTokenType type, const CSSParserToken& token)
 {
     if (type == LeftParenthesisToken) {
         if (m_mediaQueryData.restrictor() != MediaQuery::None)
             m_state = SkipUntilComma;
         else
             m_state = ReadFeature;
     } else if (type == IdentToken) {
-        if (m_state == ReadRestrictor && equalIgnoringCase(token.value(), "not")) {
+        if (m_state == ReadRestrictor && token.valueEqualsIgnoringASCIICase("not")) {
             setStateAndRestrict(ReadMediaType, MediaQuery::Not);
-        } else if (m_state == ReadRestrictor && equalIgnoringCase(token.value(), "only")) {
+        } else if (m_state == ReadRestrictor && token.valueEqualsIgnoringASCIICase("only")) {
             setStateAndRestrict(ReadMediaType, MediaQuery::Only);
         } else if (m_mediaQueryData.restrictor() != MediaQuery::None
-            && isRestrictorOrLogicalOperator(token.value())) {
+            && isRestrictorOrLogicalOperator(token)) {
             m_state = SkipUntilComma;
         } else {
             m_mediaQueryData.setMediaType(token.value());
             m_state = ReadAnd;
         }
     } else if (type == EOFToken && (!m_querySet->queryVector().size() || m_state != ReadRestrictor)) {
         m_state = Done;
     } else {
         m_state = SkipUntilComma;
         if (type == CommaToken)
             skipUntilComma(type, token);
     }
 }
 
 void MediaQueryParser::readAnd(CSSParserTokenType type, const CSSParserToken& token)
 {
-    if (type == IdentToken && equalIgnoringCase(token.value(), "and")) {
+    if (type == IdentToken && token.valueEqualsIgnoringASCIICase("and")) {
         m_state = ReadFeatureStart;
     } else if (type == CommaToken && m_parserType != MediaConditionParser) {
         m_querySet->addMediaQuery(m_mediaQueryData.takeMediaQuery());
         m_state = ReadRestrictor;
     } else if (type == EOFToken) {
         m_state = Done;
     } else {
         m_state = SkipUntilComma;
     }
 }
@@ skipping 151 matching lines @@
     return false;
 }
 
 void MediaQueryData::setMediaType(const String& mediaType)
 {
     m_mediaType = mediaType;
     m_mediaTypeSet = true;
 }
 
 } // namespace blink