Improve numeric value handling in CSS Parser

third_party/WebKit/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 "core/css/parser/CSSParserToken.h"
 
 #include "core/css/CSSMarkup.h"
 #include "core/css/CSSPrimitiveValue.h"
 #include "core/css/parser/CSSPropertyParser.h"
 #include "wtf/HashMap.h"
@@ skipping 24 matching lines @@
     initValueFromStringView(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)
 {
     ASSERT(type == NumberToken);
+    m_numericValue = clampTo<double>(numericValue, -std::numeric_limits<float>::max(), std::numeric_limits<float>::max());
 }
 
 CSSParserToken::CSSParserToken(CSSParserTokenType type, UChar32 start, UChar32 end)
     : m_type(UnicodeRangeToken)
     , m_blockType(NotBlock)
 {
     ASSERT_UNUSED(type, type == UnicodeRangeToken);
     m_unicodeRange.start = start;
     m_unicodeRange.end = end;
 }
@@ skipping 223 matching lines @@
     case RightBraceToken:
         return builder.append('}');
 
     case EOFToken:
     case CommentToken:
         ASSERT_NOT_REACHED();
         return;
     }
 }
 
-bool CSSParserToken::isValidNumericValue(double value)
-{
-    return value >= -std::numeric_limits<float>::max()
-        && value <= std::numeric_limits<float>::max();
-}
-
-bool CSSParserToken::isValidNumericValue() const
-{
-    return isValidNumericValue(numericValue());
-}
-
 } // namespace blink