Remove state-specific appendTo*/begin* methods from VTTToken

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

 /*
  * Copyright (C) 2011 Google Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 84 matching lines @@
                 if (m_token->characters().isEmpty()) {
                     WEBVTT_ADVANCE_TO(TagState);
                 } else {
                     // We don't want to advance input or perform a state transition - just return a (new) token.
                     // (On the next call to nextToken we will see '<' again, but take the other branch in this if instead.)
                     return emitToken(VTTTokenTypes::Character);
                 }
             } else if (cc == kEndOfFileMarker) {
                 return advanceAndEmitToken(source, VTTTokenTypes::Character);
             } else {
-                bufferCharacter(cc);
+                m_token->appendToData(cc);
                 WEBVTT_ADVANCE_TO(DataState);
             }
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(EscapeState) {
             if (cc == ';') {
                 if (equalLiteral(m_buffer, "&amp")) {
-                    bufferCharacter('&');
+                    m_token->appendToData('&');
                 } else if (equalLiteral(m_buffer, "&lt")) {
-                    bufferCharacter('<');
+                    m_token->appendToData('<');
                 } else if (equalLiteral(m_buffer, "&gt")) {
-                    bufferCharacter('>');
+                    m_token->appendToData('>');
                 } else if (equalLiteral(m_buffer, "&lrm")) {
-                    bufferCharacter(leftToRightMark);
+                    m_token->appendToData(leftToRightMark);
                 } else if (equalLiteral(m_buffer, "&rlm")) {
-                    bufferCharacter(rightToLeftMark);
+                    m_token->appendToData(rightToLeftMark);
                 } else if (equalLiteral(m_buffer, "&nbsp")) {
-                    bufferCharacter(noBreakSpace);
+                    m_token->appendToData(noBreakSpace);
                 } else {
                     m_buffer.append(static_cast<LChar>(cc));
-                    m_token->appendToCharacter(m_buffer);
+                    m_token->appendToData(m_buffer);
                 }
                 m_buffer.clear();
                 WEBVTT_ADVANCE_TO(DataState);
             } else if (isASCIIAlphanumeric(cc)) {
                 m_buffer.append(static_cast<LChar>(cc));
                 WEBVTT_ADVANCE_TO(EscapeState);
             } else if (cc == kEndOfFileMarker) {
-                m_token->appendToCharacter(m_buffer);
+                m_token->appendToData(m_buffer);
                 return advanceAndEmitToken(source, VTTTokenTypes::Character);
             } else {
                 if (!equalLiteral(m_buffer, "&"))
-                    m_token->appendToCharacter(m_buffer);
+                    m_token->appendToData(m_buffer);
                 m_buffer.clear();
                 WEBVTT_ADVANCE_TO(DataState);
             }
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(TagState) {
             if (isTokenizerWhitespace(cc)) {
-                m_token->beginEmptyStartTag();
+                ASSERT(m_token->name().isEmpty());
                 WEBVTT_ADVANCE_TO(StartTagAnnotationState);
             } else if (cc == '.') {
-                m_token->beginEmptyStartTag();
+                ASSERT(m_token->name().isEmpty());
                 WEBVTT_ADVANCE_TO(StartTagClassState);
             } else if (cc == '/') {
                 WEBVTT_ADVANCE_TO(EndTagState);
             } else if (WTF::isASCIIDigit(cc)) {
-                m_token->beginTimestampTag(cc);
+                m_token->appendToData(cc);
                 WEBVTT_ADVANCE_TO(TimestampTagState);
             } else if (cc == '>' || cc == kEndOfFileMarker) {
-                m_token->beginEmptyStartTag();
+                ASSERT(m_token->name().isEmpty());
                 return advanceAndEmitToken(source, VTTTokenTypes::StartTag);
             } else {
-                m_token->beginStartTag(cc);
+                m_token->appendToData(cc);
                 WEBVTT_ADVANCE_TO(StartTagState);
             }
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(StartTagState) {
             if (isTokenizerWhitespace(cc)) {
                 WEBVTT_ADVANCE_TO(StartTagAnnotationState);
             } else if (cc == '.') {
                 WEBVTT_ADVANCE_TO(StartTagClassState);
             } else if (cc == '>' || cc == kEndOfFileMarker) {
                 return advanceAndEmitToken(source, VTTTokenTypes::StartTag);
             } else {
-                m_token->appendToName(cc);
+                m_token->appendToData(cc);
                 WEBVTT_ADVANCE_TO(StartTagState);
             }
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(StartTagClassState) {
             if (isTokenizerWhitespace(cc)) {
-                m_token->addNewClass();
+                m_token->addNewClass(m_buffer);
+                m_buffer.clear();
                 WEBVTT_ADVANCE_TO(StartTagAnnotationState);
             } else if (cc == '.') {
-                m_token->addNewClass();
+                m_token->addNewClass(m_buffer);
+                m_buffer.clear();
                 WEBVTT_ADVANCE_TO(StartTagClassState);
             } else if (cc == '>' || cc == kEndOfFileMarker) {
-                m_token->addNewClass();
+                m_token->addNewClass(m_buffer);
+                m_buffer.clear();
                 return advanceAndEmitToken(source, VTTTokenTypes::StartTag);
             } else {
-                m_token->appendToClass(cc);
+                m_buffer.append(cc);
                 WEBVTT_ADVANCE_TO(StartTagClassState);
             }
-
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(StartTagAnnotationState) {
             if (cc == '>' || cc == kEndOfFileMarker) {
-                m_token->addNewAnnotation();
+                m_token->addNewAnnotation(m_buffer);
+                m_buffer.clear();
                 return advanceAndEmitToken(source, VTTTokenTypes::StartTag);
             }
-            m_token->appendToAnnotation(cc);
+            m_buffer.append(cc);
             WEBVTT_ADVANCE_TO(StartTagAnnotationState);
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(EndTagState) {
             if (cc == '>' || cc == kEndOfFileMarker)
                 return advanceAndEmitToken(source, VTTTokenTypes::EndTag);
-            m_token->appendToName(cc);
+            m_token->appendToData(cc);
             WEBVTT_ADVANCE_TO(EndTagState);
         }
         END_STATE()
 
         WEBVTT_BEGIN_STATE(TimestampTagState) {
             if (cc == '>' || cc == kEndOfFileMarker)
                 return advanceAndEmitToken(source, VTTTokenTypes::TimestampTag);
-            m_token->appendToTimestamp(cc);
+            m_token->appendToData(cc);
             WEBVTT_ADVANCE_TO(TimestampTagState);
         }
         END_STATE()
 
     }
 
     ASSERT_NOT_REACHED();
     return false;
 }
 
 }