Expired studies should not enable features.

Expired studies should not enable features.

Previosuly, the "chosen group" was queried before an
expired study was marked as enabled, which resulted in
the wrong features being activated/de-activated for
disabled studies (i.e. it would apply the ones based
on the dice roll, rather then the default group ones).

This CL fixes that and adds a test.

BUG=612533
Committed: https://crrev.com/7141106df26d989fa4a37bee08a0353fa84901ab
Cr-Commit-Position: refs/heads/master@{#394202}

[Alexei Svitkine (slow), 2016-05-17 18:53:14]

Patchset #1 (id:1) has been deleted

[Alexei Svitkine (slow), 2016-05-17 18:54:02]

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

[jwd, 2016-05-17 19:27:42]

lgtm

[Alexei Svitkine (slow), 2016-05-17 19:46:17]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2016-05-17 19:46:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1986073003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1986073003/20001

[commit-bot: I haz the power, 2016-05-17 20:10:00]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-05-17 20:11:05]

Description was changed from

==========
Expired studies should not enable features.

Previosuly, the "chosen group" was queried before an
expired study was marked as enabled, which resulted in
the wrong features being activated/de-activated for
disabled studies (i.e. it would apply the ones based
on the dice roll, rather then the default group ones).

This CL fixes that and adds a test.

BUG=612533
==========

to

==========
Expired studies should not enable features.

Previosuly, the "chosen group" was queried before an
expired study was marked as enabled, which resulted in
the wrong features being activated/de-activated for
disabled studies (i.e. it would apply the ones based
on the dice roll, rather then the default group ones).

This CL fixes that and adds a test.

BUG=612533

Committed: https://crrev.com/7141106df26d989fa4a37bee08a0353fa84901ab
Cr-Commit-Position: refs/heads/master@{#394202}
==========

[commit-bot: I haz the power, 2016-05-17 20:11:07]

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