Move WebInbandTextTrack/WebInbandTextTrackClient to public/platform/

Move WebInbandTextTrack/WebInbandTextTrackClient to public/platform/

This is to allow InbandTextTrackPrivate/InbandTextTrackPrivateClient
to be removed, by using the Web* variants directly, and for that to
be allowed they must reside in public/platform/.

Some style changes were required to please presubmit.

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

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

I've talked to Jochen about this on IRC, but it sounded like he wanted Adam to
have a look as well.

[philipj_slow, 2013-11-07 15:06:48]

See https://chromiumcodereview.appspot.com/60083011/ for more context.

[jochen (gone - plz use gerrit), 2013-11-07 15:23:12]

https://codereview.chromium.org/61763010/diff/30001/public/web/DEPS
File public/web/DEPS (right):

https://codereview.chromium.org/61763010/diff/30001/public/web/DEPS#newcode1
public/web/DEPS:1: include_rules = [
can you use specific_include_rules to restrict the includes to the two files
that actually include the headers, and add a comment that this is temporary?

[philipj_slow, 2013-11-07 15:39:31]

https://codereview.chromium.org/61763010/diff/30001/public/web/DEPS
File public/web/DEPS (right):

https://codereview.chromium.org/61763010/diff/30001/public/web/DEPS#newcode1
public/web/DEPS:1: include_rules = [
On 2013/11/07 15:23:12, jochen wrote:
> can you use specific_include_rules to restrict the includes to the two files
> that actually include the headers, and add a comment that this is temporary?

Done.

[abarth-chromium, 2013-11-07 18:25:22]

https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS
File public/web/DEPS (right):

https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS#newcode8
public/web/DEPS:8: ]
There's no need for these rules.

https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
File public/web/WebInbandTextTrack.h (right):

https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
public/web/WebInbandTextTrack.h:31: #include "../platform/WebInbandTextTrack.h"
You can just write:

#include "public/platform/WebInbandTextTrack.h"

[philipj_slow, 2013-11-07 19:42:03]

https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS
File public/web/DEPS (right):

https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS#newcode8
public/web/DEPS:8: ]
On 2013/11/07 18:25:22, abarth wrote:
> There's no need for these rules.

Removed.

https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
File public/web/WebInbandTextTrack.h (right):

https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
public/web/WebInbandTextTrack.h:31: #include "../platform/WebInbandTextTrack.h"
On 2013/11/07 18:25:22, abarth wrote:
> You can just write:
> 
> #include "public/platform/WebInbandTextTrack.h"

Done. I was tricked by the name other files in this directory using relative
paths, should those be fixed?

[philipj_slow, 2013-11-07 19:58:09]

s/name/many/

[abarth-chromium, 2013-11-07 22:02:12]

On 2013/11/07 19:42:03, philipj wrote:
> https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS
> File public/web/DEPS (right):
> 
> https://codereview.chromium.org/61763010/diff/90001/public/web/DEPS#newcode8
> public/web/DEPS:8: ]
> On 2013/11/07 18:25:22, abarth wrote:
> > There's no need for these rules.
> 
> Removed.
> 
>
https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
> File public/web/WebInbandTextTrack.h (right):
> 
>
https://codereview.chromium.org/61763010/diff/90001/public/web/WebInbandTextT...
> public/web/WebInbandTextTrack.h:31: #include
"../platform/WebInbandTextTrack.h"
> On 2013/11/07 18:25:22, abarth wrote:
> > You can just write:
> > 
> > #include "public/platform/WebInbandTextTrack.h"
> 
> Done. I was tricked by the name other files in this directory using relative
> paths, should those be fixed?

[abarth-chromium, 2013-11-07 22:03:01]

On 2013/11/07 19:42:03, philipj wrote:
> I was tricked by the name other files in this directory using relative
> paths, should those be fixed?

Yes, sorry.  I goofed it up originally.  If you're interested in fixing the
issue globally, I'm happy to review the CL.

[philipj_slow, 2013-11-07 23:34:03]

On 2013/11/07 22:03:01, abarth wrote:
> On 2013/11/07 19:42:03, philipj wrote:
> > I was tricked by the name other files in this directory using relative
> > paths, should those be fixed?
> 
> Yes, sorry.  I goofed it up originally.  If you're interested in fixing the
> issue globally, I'm happy to review the CL.

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

[jochen (gone - plz use gerrit), 2013-11-08 11:31:12]

lgtm

[commit-bot: I haz the power, 2013-11-08 13:38:59]

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

[commit-bot: I haz the power, 2013-11-08 15:21:12]

Change committed as 161626