Rename TextTrackRegion/TextTrackRegionList to VTTRegion/VTTRegionList

Rename TextTrackRegion/TextTrackRegionList to VTTRegion/VTTRegionList

http://dev.w3.org/html5/webvtt/#vttregion-interface
http://dev.w3.org/html5/webvtt/#vttregionlist-interface

This is a completely mechanical rename, except for added instanceof
tests and removing NoInterfaceObject (not per spec) from VTTRegionList
to make them pass.

Silvia (the spec editor) says in the bug that the spec hasn't changed
since the implementation, so together with the fact that this is
RuntimeEnabled, it should be safe despite being Web facing.

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

[philipj_slow, 2013-11-01 13:01:33]

Requesting review from Aaron and rubber stamping from Adam.

[vcarbune.chromium, 2013-11-01 13:06:33]

On 2013/11/01 13:01:33, philipj wrote:
> Requesting review from Aaron and rubber stamping from Adam.

FWIW, I went through the files and the changes look good, thanks!

[philipj_slow, 2013-11-01 14:36:13]

On 2013/11/01 13:06:33, vcarbune.chromium wrote:
> On 2013/11/01 13:01:33, philipj wrote:
> > Requesting review from Aaron and rubber stamping from Adam.
> 
> FWIW, I went through the files and the changes look good, thanks!

Thanks Victor!

[adamk, 2013-11-01 15:13:00]

You'll need a more powerful rubber-stamper than I (since you're touching .idl
files and thus web-exposed behavior); adding eseidel.

[acolwell GONE FROM CHROMIUM, 2013-11-01 19:32:48]

lgtm

https://codereview.chromium.org/55653003/diff/1/Source/core/html/track/VTTReg...
File Source/core/html/track/VTTRegionList.idl (left):

https://codereview.chromium.org/55653003/diff/1/Source/core/html/track/VTTReg...
Source/core/html/track/VTTRegionList.idl:29: RuntimeEnabled=WebVTTRegions
Consider adding a , depends_on=VideoTrack for WebVTTRegions in
Source/platform/RuntimeEnabledFeatures.in and removing the
RuntimeEnabled=VideoTrack here and in the other IDL in a follow-up CL. I believe
that will provide a stronger guarentee that the region code won't be enabled if
VideoTrack is not enabled.

[ojan, 2013-11-01 22:05:14]

lgtm

[philipj_slow, 2013-11-01 22:06:39]

https://codereview.chromium.org/55653003/diff/1/Source/core/html/track/VTTReg...
File Source/core/html/track/VTTRegionList.idl (left):

https://codereview.chromium.org/55653003/diff/1/Source/core/html/track/VTTReg...
Source/core/html/track/VTTRegionList.idl:29: RuntimeEnabled=WebVTTRegions
On 2013/11/01 19:32:49, acolwell wrote:
> Consider adding a , depends_on=VideoTrack for WebVTTRegions in
> Source/platform/RuntimeEnabledFeatures.in and removing the
> RuntimeEnabled=VideoTrack here and in the other IDL in a follow-up CL. I
believe
> that will provide a stronger guarentee that the region code won't be enabled
if
> VideoTrack is not enabled.

Done in https://codereview.chromium.org/54903008/

[commit-bot: I haz the power, 2013-11-01 22:07:07]

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

[commit-bot: I haz the power, 2013-11-01 23:37:59]

Change committed as 161209