Add a control group to the SiteIsolationExtensionsActive. It corresponds to the Disabled group of t…

Add a control group to the SiteIsolationExtensionsActive.

"Control" corresponds to the Disabled group of the [non-synthetic]
SiteIsolationExtensions field trial. In practice, our analysis
has been skewed by the fact that SiteIsolationExtensionsActive/Default
includes samples from populations where the SiteIsolationExtensions trial was
not operative; these samples were a source of confounding variables.

BUG=542921
Committed: https://crrev.com/1b39cd77e71f4c7d64e2ab4c1b0d3336f898514c
Cr-Commit-Position: refs/heads/master@{#375027}

[ncarter (slow), 2016-02-11 19:44:38]

nick@chromium.org changed reviewers:
+ creis@chromium.org, thestig@chromium.org

[ncarter (slow), 2016-02-11 19:44:38]

charlie & lei, PTAL

[Charlie Reis, 2016-02-11 21:19:01]

Thanks, LGTM.   This will let us exclude OOPIF-able users who aren't in the
SiteIsolationExtensions trial from our data analysis, since that's affecting the
results.

[Charlie Reis, 2016-02-11 21:19:23]

[+site-isolation-reviews for FYI]

[Lei Zhang, 2016-02-11 21:26:48]

lgtm

[ncarter (slow), 2016-02-11 21:31:01]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-02-11 21:34:12]

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

[commit-bot: I haz the power, 2016-02-11 21:34:12]

This CL has an open dependency (Issue 1693583003 Patch 1). Please resolve the
dependency and try again.

[ncarter (slow), 2016-02-11 22:53:52]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2016-02-11 22:54:32]

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

[commit-bot: I haz the power, 2016-02-11 23:08:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-16 22:39:08]

Description was changed from

==========
Add a control group to the SiteIsolationExtensionsActive.

"Control" corresponds to the Disabled group of the [non-synthetic]
SiteIsolationExtensions field trial. In practice, our analysis
has been skewed by the fact that SiteIsolationExtensionsActive/Default
includes samples from populations where the SiteIsolationExtensions trial was
not operative; these samples were a source of confounding variables.

BUG=542921
==========

to

==========
Add a control group to the SiteIsolationExtensionsActive.

"Control" corresponds to the Disabled group of the [non-synthetic]
SiteIsolationExtensions field trial. In practice, our analysis
has been skewed by the fact that SiteIsolationExtensionsActive/Default
includes samples from populations where the SiteIsolationExtensions trial was
not operative; these samples were a source of confounding variables.

BUG=542921

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

[commit-bot: I haz the power, 2016-02-16 22:39:10]

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