Rework end-of-value field handling in VTTRegion::parseSettingValue

Source/core/html/track/vtt/VTTParser.cpp

Index: Source/core/html/track/vtt/VTTParser.cpp
diff --git a/Source/core/html/track/vtt/VTTParser.cpp b/Source/core/html/track/vtt/VTTParser.cpp
index 6148ea67bb57a9757104b12869ba555d2601c875..6cb7c45f807bdc580d0cef8e61008d7730677211 100644
--- a/Source/core/html/track/vtt/VTTParser.cpp
+++ b/Source/core/html/track/vtt/VTTParser.cpp
@@ -48,12 +48,6 @@ const double secondsPerMinute = 60;
 const double secondsPerMillisecond = 0.001;
 const unsigned fileIdentifierLength = 6;
 
-unsigned VTTParser::collectDigitsToInt(const String& input, unsigned* position, int& number)
-{
-    VTTLegacyScanner inputScanner(input, position);
-    return inputScanner.scanDigits(number);
-}
-
 bool VTTParser::parseFloatPercentageValue(VTTScanner& valueScanner, float& percentage)
 {
     float number;
@@ -68,10 +62,8 @@ bool VTTParser::parseFloatPercentageValue(VTTScanner& valueScanner, float& perce
     return true;
 }
 
-bool VTTParser::parseFloatPercentageValuePair(const String& value, char delimiter, FloatPoint& valuePair)
+bool VTTParser::parseFloatPercentageValuePair(VTTScanner& valueScanner, char delimiter, FloatPoint& valuePair)
 {
-    VTTScanner valueScanner(value);
-
     float firstCoord;
     if (!parseFloatPercentageValue(valueScanner, firstCoord))
         return false;
@@ -83,9 +75,6 @@ bool VTTParser::parseFloatPercentageValuePair(const String& value, char delimite
     if (!parseFloatPercentageValue(valueScanner, secondCoord))
         return false;
 
-    if (!valueScanner.isAtEnd())
-        return false;
-
     valuePair = FloatPoint(firstCoord, secondCoord);
     return true;
 }