Cleanups for VTTTokenizer

Cleanups for VTTTokenizer

Get rid of VTTTokenizer::m_token, and hence remove the need to play
games with pointers to the resulting token, since the token is now
produced at the end of VTTTokenizer::nextToken. This renders the reset()
method empty (and it wasn't previously used outside of VTTTokenizer), so
remove it.

Make a local, tailored, version of the ADVANCE_TO macro to get rid of the
redundant check of advance(). This in turn allows removing the (now
useless) part of the scaffolding which is haveBufferedCharacterToken(),
as well as turning VTTTokenizer::m_state into the local 'state' and
moving the definition of the state enumeration into the cpp-file.
This also means the state defintions can be moved from the header-file,
so do that as well.

BUG=319391
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=162994

[Mike West, 2013-12-02 14:45:50]

This looks significantly simpler than the current setup, and means that we don't
have to hold a reference to a piece of memory we don't own. LGTM, thanks.

[commit-bot: I haz the power, 2013-12-02 15:10:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/98763004/1

[commit-bot: I haz the power, 2013-12-02 16:08:14]

Change committed as 162994