Correct use of virtual and OVERRIDE in HTMLTrackElement

Correct use of virtual and OVERRIDE in HTMLTrackElement

canLoadUrl was virtual for no reason, and the TextTrackClient
implementation was missing OVERRIDE.

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

[eseidel, 2013-10-30 22:54:43]

Maybe it used to be part of TextTrackClient?
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[Nate Chapin, 2013-10-30 23:15:26]

lgtm

[philipj_slow, 2013-10-31 07:33:10]

On 2013/10/30 22:54:43, eseidel wrote:
> Maybe it used to be part of TextTrackClient?
>
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

It was added in <https://trac.webkit.org/changeset/101999> and judging by that
context, the original intention must have been for didCompleteLoad and
canLoadUrl to be part of some interface, but even at the time of that change it
wasn't actually the case.

[commit-bot: I haz the power, 2013-10-31 07:33:37]

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

[commit-bot: I haz the power, 2013-10-31 09:42:43]

Retried try job too often on linux_blink for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2013-10-31 09:59:58]

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

[commit-bot: I haz the power, 2013-10-31 10:39:57]

Change committed as 161047