Use TextTrackMode and TextTrackKind enums in TextTrack IDL

Use TextTrackMode and TextTrackKind enums in TextTrack IDL

The generated code (modeAttributeSetter in V8TextTrack.cpp) now
checks that the string is one of the three allowed values, so we
can just assert it internally.

Also fiddle a bit with the IDL to make it match the spec order.

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

[jochen (gone - plz use gerrit), 2013-10-31 10:32:10]

lgtm

https://codereview.chromium.org/52593009/diff/1/Source/core/html/track/TextTr...
File Source/core/html/track/TextTrack.idl (right):

https://codereview.chromium.org/52593009/diff/1/Source/core/html/track/TextTr...
Source/core/html/track/TextTrack.idl:44: attribute EventHandler oncuechange;
why did you move the attribute down here?

[philipj_slow, 2013-10-31 10:40:13]

https://codereview.chromium.org/52593009/diff/1/Source/core/html/track/TextTr...
File Source/core/html/track/TextTrack.idl (right):

https://codereview.chromium.org/52593009/diff/1/Source/core/html/track/TextTr...
Source/core/html/track/TextTrack.idl:44: attribute EventHandler oncuechange;
On 2013/10/31 10:32:10, jochen wrote:
> why did you move the attribute down here?

To match the order of the spec:
http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element...

This makes it easier to eyeball diff the spec and implementation.

[commit-bot: I haz the power, 2013-10-31 10:47:54]

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

[commit-bot: I haz the power, 2013-10-31 12:58:56]

Change committed as 161055