CSS Properties and Values API: Reject CSS-wide keywords in syntax string

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

 // Copyright 2016 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/CSSSyntaxDescriptor.h"
 
 #include "core/css/CSSCustomPropertyDeclaration.h"
 #include "core/css/CSSURIValue.h"
 #include "core/css/CSSValueList.h"
 #include "core/css/CSSVariableReferenceValue.h"
@@ skipping 63 matching lines @@
         return false;
     type = parseSyntaxType(input.substring(typeStart, offset - typeStart));
     if (type == CSSSyntaxType::Ident)
         return false;
     offset++;
     return true;
 }
 
 bool consumeSyntaxIdent(const String& input, size_t& offset, String& ident)
 {
-    // TODO(timloh): Are CSS-wide keywords allowed here?
     size_t identStart = offset;
     while (isNameCodePoint(input[offset]))
         offset++;
     if (offset == identStart)
         return false;
     ident = input.substring(identStart, offset - identStart);
-    return true;
+    return !CSSPropertyParserHelpers::isCSSWideKeyword(ident);
 }
 
 CSSSyntaxDescriptor::CSSSyntaxDescriptor(String input)
 {
     size_t offset = 0;
     consumeWhitespace(input, offset);
 
     if (consumeCharacterAndWhitespace(input, '*', offset)) {
         if (offset != input.length())
             return;
@@ skipping 99 matching lines @@
         return CSSVariableParser::parseRegisteredPropertyValue(range, false);
     range.consumeWhitespace();
     for (const CSSSyntaxComponent& component : m_syntaxComponents) {
         if (const CSSValue* result = consumeSyntaxComponent(component, range))
             return result;
     }
     return CSSVariableParser::parseRegisteredPropertyValue(range, true);
 }
 
 } // namespace blink