Add a flag for enabling/disabling the navigator content utils

Add a flag for enabling/disabling the navigator content utils

As a part of removing ENABLE(NAVIGATOR_CONTENT_UTILS), this cl add a flag for the navigator content utils
and enable it except for android.

BUG=N/A

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=239691

[abarth-chromium, 2013-12-04 17:11:27]

I'm not an owner here.  You'll have to ask someone else.

[gyuyoung-inactive, 2013-12-05 00:18:34]

Add jochen and darin. Could you take a look this cl ? 

BTW, abarth, 91313002 should be landed first in order to land this cl. Could you
land it first ?

[jochen (gone - plz use gerrit), 2013-12-05 08:57:41]

lgtm

[gyuyoung-inactive, 2013-12-05 13:24:47]

On 2013/12/05 08:57:41, jochen wrote:
> lgtm

Abarth, could you take a look 91313002 ? It needs to be landed first.

[gyuyoung-inactive, 2013-12-09 11:31:43]

On 2013/12/05 13:24:47, gyuyoung wrote:
> On 2013/12/05 08:57:41, jochen wrote:
> > lgtm
> 
> Abarth, could you take a look 91313002 ? It needs to be landed first.

Jochen, issue 91313002 was landed. So, this cl need to be landed now. Then,
issue 110183002 will remove #if ENABLE(NAVIGATOR_CONTENT_UTILS) finally. 

Could you land this cl ?

[jochen (gone - plz use gerrit), 2013-12-09 12:16:10]

On 2013/12/09 11:31:43, gyuyoung wrote:
> On 2013/12/05 13:24:47, gyuyoung wrote:
> > On 2013/12/05 08:57:41, jochen wrote:
> > > lgtm
> > 
> > Abarth, could you take a look 91313002 ? It needs to be landed first.
> 
> Jochen, issue 91313002 was landed. So, this cl need to be landed now. Then,
> issue 110183002 will remove #if ENABLE(NAVIGATOR_CONTENT_UTILS) finally. 
> 
> Could you land this cl ?

you can just check the Commit: checkbox (I did that for you now)

[commit-bot: I haz the power, 2013-12-09 12:16:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/104123003/1

[commit-bot: I haz the power, 2013-12-09 12:29:42]

Retried try job too often on linux_rel for step(s) base_unittests,
browser_tests, interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[jochen (gone - plz use gerrit), 2013-12-09 12:32:20]

I guess the blink side changed hasn't yet rolled in.

once it has rolled in, just check the flag again

[jam, 2013-12-09 16:35:56]

https://codereview.chromium.org/104123003/diff/1/content/public/common/conten...
File content/public/common/content_switches.h (right):

https://codereview.chromium.org/104123003/diff/1/content/public/common/conten...
content/public/common/content_switches.h:57: extern const char
kDisableNavigatorContentUtils[];
nit: note this file is sorted, so put this in the right place

[gyuyoung-inactive, 2013-12-10 00:38:15]

On 2013/12/09 16:35:56, jam wrote:
>
https://codereview.chromium.org/104123003/diff/1/content/public/common/conten...
> File content/public/common/content_switches.h (right):
> 
>
https://codereview.chromium.org/104123003/diff/1/content/public/common/conten...
> content/public/common/content_switches.h:57: extern const char
> kDisableNavigatorContentUtils[];
> nit: note this file is sorted, so put this in the right place

Fixed.

[gyuyoung-inactive, 2013-12-10 00:42:00]

On 2013/12/09 12:32:20, jochen wrote:
> I guess the blink side changed hasn't yet rolled in.
> 
> once it has rolled in, just check the flag again

Blink patch was landed yesterday.
https://src.chromium.org/viewvc/blink?view=rev&revision=163413

I'd like to test new patch again.

[gyuyoung-inactive, 2013-12-10 00:50:01]

On 2013/12/09 12:16:10, jochen wrote:
> On 2013/12/09 11:31:43, gyuyoung wrote:
> > On 2013/12/05 13:24:47, gyuyoung wrote:
> > > On 2013/12/05 08:57:41, jochen wrote:
> > > > lgtm
> > > 
> > > Abarth, could you take a look 91313002 ? It needs to be landed first.
> > 
> > Jochen, issue 91313002 was landed. So, this cl need to be landed now. Then,
> > issue 110183002 will remove #if ENABLE(NAVIGATOR_CONTENT_UTILS) finally. 
> > 
> > Could you land this cl ?
> 
> you can just check the Commit: checkbox (I did that for you now)

I'm not sure if I have a power to check the Commit. Anyway, now I try it.

[commit-bot: I haz the power, 2013-12-10 01:25:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/104123003/20001

[gyuyoung-inactive, 2013-12-10 05:08:10]

On 2013/12/10 01:25:32, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>
https://chromium-status.appspot.com/cq/gyuyoung.kim%40samsung.com/104123003/2...

Jochen, it looks this cl pass all ews bots. However, it looks I don't have any
power to land this cl. Please land this cl.

[commit-bot: I haz the power, 2013-12-10 07:13:16]

Change committed as 239691