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

Source/core/html/track/vtt/VTTScanner.h

 /*
  * Copyright (c) 2013, Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 210 matching lines @@
 
 inline void VTTScanner::advance(unsigned amount)
 {
     ASSERT(position() < end());
     if (m_is8Bit)
         m_data.characters8 += amount;
     else
         m_data.characters16 += amount;
 }
 
-// Wrapper of VTTScanner that allows easy interaction with a parsing model
-// where a <String, index> tuple is used.
-class VTTLegacyScanner : public VTTScanner {
-    WTF_MAKE_NONCOPYABLE(VTTLegacyScanner);
-public:
-    VTTLegacyScanner(const String& line, unsigned* outPosition)
-        : VTTScanner(line)
-        , m_outPosition(outPosition)
-    {
-        ASSERT(outPosition && *outPosition <= line.length());
-        // Adjust state according to |*outPosition|.
-        advance(*outPosition);
-        // Save the start position to allow adjusting |*outPosition|.
-        if (m_is8Bit)
-            m_start.characters8 = m_data.characters8;
-        else
-            m_start.characters16 = m_data.characters16;
-    }
-    ~VTTLegacyScanner()
-    {
-        // "Export" the updated position.
-        unsigned advancedChars = m_is8Bit ? m_data.characters8 - m_start.characters8 : m_data.characters16 - m_start.characters16;
-        *m_outPosition += advancedChars;
-    }
-
-private:
-    unsigned* m_outPosition;
-    union {
-        const LChar* characters8;
-        const UChar* characters16;
-    } m_start;
-};
-
 }
 
 #endif