Add base:: to string16 in Source/testing/runner/.

Add base:: to string16 in Source/testing/runner/.

R=eseidel@chromium.org
BUG=329295

[eseidel, 2013-12-19 06:15:46]

Why?  We really shouldn't be using base in Blink anyway...

[eseidel, 2013-12-19 06:16:12]

Is there a "using namespace base" which we should be removing here?

[Dan Beam, 2013-12-19 06:31:36]

On 2013/12/19 06:16:12, eseidel wrote:
> Is there a "using namespace base" which we should be removing here?

it's at the end of base/strings/string16.h and can be removed only after all
`string16`s are made `base::string16`.

https://code.google.com/p/chromium/codesearch#chromium/src/base/strings/strin...

[eseidel, 2013-12-19 06:40:00]

lgtm

OK.  An alternate fix would be to just make these not be base::string16's at all
(since Blink really shouldn't depend on base/ yet).

But OK.

[Dan Beam, 2013-12-20 05:22:53]

On 2013/12/19 06:40:00, eseidel wrote:
> lgtm
> 
> OK.  An alternate fix would be to just make these not be base::string16's at
all
> (since Blink really shouldn't depend on base/ yet).

I may attempt to do this instead ^

> 
> But OK.

[eseidel, 2013-12-20 05:30:57]

Jochen is moving this to content, btw.  I think you want to just wait for him to
be done.

[jochen (gone - plz use gerrit), 2013-12-20 14:58:06]

On 2013/12/20 05:30:57, eseidel wrote:
> Jochen is moving this to content, btw.  I think you want to just wait for him
to
> be done.

the library is now in content/shell/renderer/test_runner

[Dan Beam, 2013-12-20 19:01:12]

On 2013/12/20 14:58:06, jochen_ooo wrote:
> On 2013/12/20 05:30:57, eseidel wrote:
> > Jochen is moving this to content, btw.  I think you want to just wait for
him
> to
> > be done.
> 
> the library is now in content/shell/renderer/test_runner

great, here's the CL to update it (closing this one):
https://codereview.chromium.org/113653003/