Add property parser code path based on CSSParserTokenRange

Source/core/css/parser/CSSParserToken.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 "config.h"
 #include "core/css/parser/CSSParserToken.h"
 
 #include "core/css/CSSMarkup.h"
 #include "core/css/parser/CSSPropertyParser.h"
 #include "wtf/HashMap.h"
@@ skipping 15 matching lines @@
     , m_delimiter(c)
 {
     ASSERT(m_type == DelimiterToken);
 }
 
 CSSParserToken::CSSParserToken(CSSParserTokenType type, CSSParserString value, BlockType blockType)
     : m_type(type)
     , m_blockType(blockType)
 {
     initValueFromCSSParserString(value);
+    m_id = -1;
 }
 
 CSSParserToken::CSSParserToken(CSSParserTokenType type, double numericValue, NumericValueType numericValueType, NumericSign sign)
     : m_type(type)
     , m_blockType(NotBlock)
     , m_numericValueType(numericValueType)
     , m_numericSign(sign)
     , m_unit(static_cast<unsigned>(CSSPrimitiveValue::UnitType::Number))
     , m_numericValue(numericValue)
 {
@@ skipping 57 matching lines @@
     ASSERT(m_type == NumberToken || m_type == PercentageToken || m_type == DimensionToken);
     return m_numericValue;
 }
 
 CSSPropertyID CSSParserToken::parseAsUnresolvedCSSPropertyID() const
 {
     ASSERT(m_type == IdentToken);
     return unresolvedCSSPropertyID(value());
 }
 
+CSSValueID CSSParserToken::id() const
+{
+    if (m_type != IdentToken && m_type != FunctionToken)

[Timothy Loh, 2015/09/10 01:13:37]

Can we split this into id() and functionId()? Otherwise something like "page:
auto(" would be accepted.

[rwlbuis, 2015/09/10 21:52:41]

Good idea, done. I also changed a few places outside of the property parser to
use these.

+        return CSSValueInvalid;
+    if (m_id < 0)
+        m_id = cssValueKeywordID(value());
+    return static_cast<CSSValueID>(m_id);
+}
+
 void CSSParserToken::serialize(StringBuilder& builder) const
 {
     // This is currently only used for @supports CSSOM. To keep our implementation
     // simple we handle some of the edge cases incorrectly (see comments below).
     switch (type()) {
     case IdentToken:
         serializeIdentifier(value(), builder);
         break;
     case FunctionToken:
         serializeIdentifier(value(), builder);
@@ skipping 74 matching lines @@
         return builder.append('}');
 
     case EOFToken:
     case CommentToken:
         ASSERT_NOT_REACHED();
         return;
     }
 }
 
 } // namespace blink