Omnibox: Launch URL Demotion in the Fakebox by Default

Omnibox: Launch URL Demotion in the Fakebox by Default

This has been approved for launch.

BUG=

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

[Mark P, 2014-02-27 22:36:59]

Please take a look, thanks.

[Peter Kasting, 2014-02-27 22:38:27]

LGTM; does "approved for launch" mean "we saw positive results from our trials"?
 I'm curious about what effects this has.

[Mark P, 2014-02-27 23:02:55]

On 2014/02/27 22:38:27, Peter Kasting wrote:
> LGTM; does "approved for launch" mean "we saw positive results from our
trials"?
>  I'm curious about what effects this has.

Answered via e-mail.

--mark

[H Fung, 2014-02-27 23:38:38]

Thanks, but I'm not sure if you want the server-side change (to prevent queries
from being inlined when there's URL intent on fakebox context) to roll out
before you submit.

[Mark P, 2014-02-27 23:43:44]

On 2014/02/27 23:38:38, H Fung wrote:
> Thanks, but I'm not sure if you want the server-side change (to prevent
queries
> from being inlined when there's URL intent on fakebox context) to roll out
> before you submit.

Okay, I will wait until you tell me the server-side change is out.

--mark

[H Fung, 2014-03-03 19:41:35]

On 2014/02/27 23:43:44, Mark P wrote:
> On 2014/02/27 23:38:38, H Fung wrote:
> > Thanks, but I'm not sure if you want the server-side change (to prevent
> queries
> > from being inlined when there's URL intent on fakebox context) to roll out
> > before you submit.
> 
> Okay, I will wait until you tell me the server-side change is out.
> 
> --mark

The server-side change should be out this Wednesday, so lgtm.

[Mark P, 2014-03-06 23:34:41]

The CQ bit was checked by mpearson@chromium.org

[commit-bot: I haz the power, 2014-03-06 23:45:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpearson@chromium.org/182313014/1

[commit-bot: I haz the power, 2014-03-07 13:45:53]

Change committed as 255606