Adding support for exclude locales to variations.

Adding support for exclude locales to variations.

BUG=664682
Committed: https://crrev.com/1db68a10877c1732b26473581ce0b8866be6fbda
Cr-Commit-Position: refs/heads/master@{#431722}

[jwd, 2016-11-11 20:22:23]

The CQ bit was checked by jwd@chromium.org to run a CQ dry run

[jwd, 2016-11-11 20:22:38]

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

[commit-bot: I haz the power, 2016-11-11 20:22:48]

Dry run: 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, 2016-11-11 21:03:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-11 21:03:31]

Dry run: This issue passed the CQ dry run.

[rkaplow, 2016-11-11 21:21:10]

lgtm

nit, you probably should have a crbug associated

https://codereview.chromium.org/2488173005/diff/1/components/variations/study...
File components/variations/study_filtering.cc (right):

https://codereview.chromium.org/2488173005/diff/1/components/variations/study...
components/variations/study_filtering.cc:113: // Check if we are spposed to
filter for a specified set of countries. Note
supposed

[jwd, 2016-11-12 00:14:05]

Description was changed from

==========
Adding support for exclude locales to variations.

BUG=
==========

to

==========
Adding support for exclude locales to variations.

BUG=664682
==========

[jwd, 2016-11-12 00:21:14]

https://codereview.chromium.org/2488173005/diff/1/components/variations/study...
File components/variations/study_filtering.cc (right):

https://codereview.chromium.org/2488173005/diff/1/components/variations/study...
components/variations/study_filtering.cc:113: // Check if we are spposed to
filter for a specified set of countries. Note
On 2016/11/11 21:21:10, rkaplow wrote:
> supposed

Done.

[jwd, 2016-11-12 00:21:24]

The CQ bit was checked by jwd@chromium.org

[jwd, 2016-11-12 00:21:24]

The patchset sent to the CQ was uploaded after l-g-t-m from rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2488173005/#ps20001
(title: "fix typo in comment")

[commit-bot: I haz the power, 2016-11-12 00:22:16]

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, 2016-11-12 01:23:19]

Description was changed from

==========
Adding support for exclude locales to variations.

BUG=664682
==========

to

==========
Adding support for exclude locales to variations.

BUG=664682
==========

[commit-bot: I haz the power, 2016-11-12 01:23:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-12 01:26:15]

Description was changed from

==========
Adding support for exclude locales to variations.

BUG=664682
==========

to

==========
Adding support for exclude locales to variations.

BUG=664682

Committed: https://crrev.com/1db68a10877c1732b26473581ce0b8866be6fbda
Cr-Commit-Position: refs/heads/master@{#431722}
==========

[commit-bot: I haz the power, 2016-11-12 01:26:16]

Patchset 2 (id:??) landed as
https://crrev.com/1db68a10877c1732b26473581ce0b8866be6fbda
Cr-Commit-Position: refs/heads/master@{#431722}