Add Finch testing variations for search geolocation disclosure.

Add Finch testing variations for search geolocation disclosure.

BUG=661011
Review-Url: https://codereview.chromium.org/2626073006
Cr-Commit-Position: refs/heads/master@{#443447}
Committed: https://chromium.googlesource.com/chromium/src/+/00cd9cf3c6e1b0dbc637090df6bf567e043c3c5b

[benwells, 2017-01-12 04:32:15]

benwells@chromium.org changed reviewers:
+ dominickn@chromium.org

[benwells, 2017-01-12 04:32:15]

dom for local sanity check

[dominickn, 2017-01-12 04:45:06]

lgtm

FYI: only the first config will be run on the perf test bots, so make sure it's
the one most likely to launch.

[benwells, 2017-01-12 04:58:08]

On 2017/01/12 04:45:06, dominickn wrote:
> lgtm
> 
> FYI: only the first config will be run on the perf test bots, so make sure
it's
> the one most likely to launch.

Yep, it's the one we'll probably go with.

[benwells, 2017-01-12 04:58:29]

benwells@chromium.org changed reviewers:
+ rkaplow@chromium.org

[benwells, 2017-01-12 04:58:30]

+rkaplow for owners review

[benwells, 2017-01-12 07:22:29]

Description was changed from

==========
Add Finch testing variations for search geolocation disclosure.

BUG=661011
==========

to

==========
Add Finch testing variations for search geolocation disclosure.

BUG=661011
==========

[benwells, 2017-01-12 07:22:42]

+cc raymes

[rkaplow, 2017-01-12 16:30:36]

https://codereview.chromium.org/2626073006/diff/20001/testing/variations/fiel...
File testing/variations/fieldtrial_testing_config.json (right):

https://codereview.chromium.org/2626073006/diff/20001/testing/variations/fiel...
testing/variations/fieldtrial_testing_config.json:2062: "experiments": [
we only support one experiment tested in the test bots so we recommend usually
just having one which you want to get tested. That group should be the
experiment group you think is most likely to be released.

[rkaplow, 2017-01-12 16:45:17]

lgtm

[benwells, 2017-01-12 23:29:38]

https://codereview.chromium.org/2626073006/diff/20001/testing/variations/fiel...
File testing/variations/fieldtrial_testing_config.json (right):

https://codereview.chromium.org/2626073006/diff/20001/testing/variations/fiel...
testing/variations/fieldtrial_testing_config.json:2062: "experiments": [
On 2017/01/12 16:30:36, rkaplow wrote:
> we only support one experiment tested in the test bots so we recommend usually
> just having one which you want to get tested. That group should be the
> experiment group you think is most likely to be released.

Ah, OK. I've removed all but the first one, which is the one we think will be
released.

[benwells, 2017-01-12 23:29:43]

The CQ bit was checked by benwells@chromium.org

[benwells, 2017-01-12 23:29:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
dominickn@chromium.org, rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2626073006/#ps40001
(title: "Remove all but first experiments")

[benwells, 2017-01-12 23:29:44]

The CQ bit was unchecked by benwells@chromium.org

[benwells, 2017-01-12 23:29:50]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2017-01-12 23:30:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-13 01:45:44]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484263790646990,
"parent_rev": "4ebe01d1636b1ed0b258584b2485470646f7aec9", "commit_rev":
"00cd9cf3c6e1b0dbc637090df6bf567e043c3c5b"}

[commit-bot: I haz the power, 2017-01-13 01:46:14]

Description was changed from

==========
Add Finch testing variations for search geolocation disclosure.

BUG=661011
==========

to

==========
Add Finch testing variations for search geolocation disclosure.

BUG=661011

Review-Url: https://codereview.chromium.org/2626073006
Cr-Commit-Position: refs/heads/master@{#443447}
Committed:
https://chromium.googlesource.com/chromium/src/+/00cd9cf3c6e1b0dbc637090df6bf...
==========

[commit-bot: I haz the power, 2017-01-13 01:46:21]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/00cd9cf3c6e1b0dbc637090df6bf...