Remove InbandTextTrackPrivate/InbandTextTrackPrivateClient

Remove InbandTextTrackPrivate/InbandTextTrackPrivateClient

These interfaces were virtually identical to the Web* variants, so
just use those directly.

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

[philipj_slow, 2013-11-07 14:13:32]

Aaron, can you have a look at this before I bring in core owners?

Please scrutinize the lifetime and ownership of the objects closely, because
while it looks simple (the platform owns WebInbandTextTrack and guarantees that
the pointer will live until the remove call) there might be something I missed.

There are two additional problems that need to be resolved:

1. Even if I break stuff intentionally, no tests fail. Is there any test
coverage of in-band tracks at all via LayoutTests, or is there no support until
<https://codereview.chromium.org/23702007/> lands?

2. The presubmit hook is not happy about including from public/web/ inside
Source/core, but how else can I do this CL or <http://crbug.com/310466>?

[philipj_slow, 2013-11-07 15:13:12]

On 2013/11/07 14:13:32, philipj wrote:

> 2. The presubmit hook is not happy about including from public/web/ inside
> Source/core, but how else can I do this CL or <http://crbug.com/310466%3E?

I've solved this with some help from Jochen, this review now depends on
https://codereview.chromium.org/61763010/ to land first.

[acolwell GONE FROM CHROMIUM, 2013-11-07 18:07:36]

lgtm. Thanks for doing this.

[philipj_slow, 2013-11-08 13:40:33]

Jochen, can you take a look while the Californians sleep?

[jochen (gone - plz use gerrit), 2013-11-08 13:51:41]

lgtm

[commit-bot: I haz the power, 2013-11-08 15:53:52]

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

[commit-bot: I haz the power, 2013-11-08 17:34:23]

Change committed as 161632