Bug 1390703 - Bump Flash Click-to-Play rollout to 25% on release. r=bsmedberg
Also bump it on Beta to 100% as we'll do the same on release shortly afterwards.
MozReview-Commit-ID: 2Ro2agu2NY8
--- a/browser/extensions/clicktoplay-rollout/bootstrap.js
+++ b/browser/extensions/clicktoplay-rollout/bootstrap.js
@@ -9,18 +9,18 @@ const {classes: Cc, interfaces: Ci, util
Cu.import("resource://gre/modules/Preferences.jsm");
Cu.import("resource://gre/modules/Services.jsm");
Cu.import("resource://gre/modules/UpdateUtils.jsm");
Cu.import("resource://gre/modules/AppConstants.jsm");
Cu.import("resource://gre/modules/TelemetryEnvironment.jsm");
// The amount of people to be part of the rollout
const TEST_THRESHOLD = {
- "beta": 0.5, // 50%
- "release": 0.05, // 5%
+ "beta": 1.0, // 100%
+ "release": 0.25, // 25%
};
if (AppConstants.RELEASE_OR_BETA) {
// The rollout is controlled by the channel name, which
// is the only way to distinguish between Beta and Release.
// However, non-official release builds (like the ones done by distros
// to ship Firefox on their package managers) do not set a value
// for the release channel, which gets them to the default value
@@ -68,16 +68,17 @@ function defineCohort() {
break;
}
}
switch (cohort) {
case undefined:
case "test":
case "control":
+ case "excluded":
{
// If it's either test/control, the cohort might have changed
// if the desired sampling has been changed.
let testThreshold = TEST_THRESHOLD[updateChannel];
let userSample = getUserSample();
if (userSample < testThreshold) {
cohort = "test";
--- a/browser/extensions/clicktoplay-rollout/install.rdf.in
+++ b/browser/extensions/clicktoplay-rollout/install.rdf.in
@@ -5,17 +5,17 @@
#filter substitution
<RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
xmlns:em="http://www.mozilla.org/2004/em-rdf#">
<Description about="urn:mozilla:install-manifest">
<em:id>clicktoplay-rollout@mozilla.org</em:id>
- <em:version>1.2</em:version>
+ <em:version>1.3</em:version>
<em:type>2</em:type>
<em:bootstrap>true</em:bootstrap>
<em:multiprocessCompatible>true</em:multiprocessCompatible>
<!-- Target Application this theme can install into,
with minimum and maximum supported versions. -->
<em:targetApplication>
<Description>