Bug 1314429 - Allow every add-on to run on e10s, except those explictly marked with multiprocessCompatible=false. r=mconley a=gchang
authorFelipe Gomes <felipc@gmail.com>
Wed, 30 Nov 2016 20:20:20 -0200
changeset 359020 fda168823ea2e8102ded10dbb3de97975826a50e
parent 359019 5d58bac871bbca24e89effcf099624644e791000
child 359021 9c9278793a1526d65c5cc3437d030d880f14cfe2
push id1324
push usermtabara@mozilla.com
push dateMon, 16 Jan 2017 13:07:44 +0000
treeherdermozilla-release@a01c49833940 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmconley, gchang
bugs1314429
milestone51.0
Bug 1314429 - Allow every add-on to run on e10s, except those explictly marked with multiprocessCompatible=false. r=mconley a=gchang MozReview-Commit-ID: 2Jm1gtbnlxs
browser/extensions/e10srollout/bootstrap.js
toolkit/mozapps/extensions/internal/E10SAddonsRollout.jsm
--- a/browser/extensions/e10srollout/bootstrap.js
+++ b/browser/extensions/e10srollout/bootstrap.js
@@ -12,17 +12,17 @@ Cu.import("resource://gre/modules/Update
 
  // The amount of people to be part of e10s
 const TEST_THRESHOLD = {
   "beta"    : 0.5,  // 50%
   "release" : 1.0,  // 100%
 };
 
 const ADDON_ROLLOUT_POLICY = {
-  "beta"    : "50allmpc", // Any WebExtension or addon with mpc = true
+  "beta"    : "51alladdons", // Any WebExtension or addon with mpc = true
   "release" : "50allmpc", // Any WebExtension or addon with mpc = true
 };
 
 const PREF_COHORT_SAMPLE       = "e10s.rollout.cohortSample";
 const PREF_COHORT_NAME         = "e10s.rollout.cohort";
 const PREF_E10S_OPTED_IN       = "browser.tabs.remote.autostart";
 const PREF_E10S_FORCE_ENABLED  = "browser.tabs.remote.force-enable";
 const PREF_E10S_FORCE_DISABLED = "browser.tabs.remote.force-disable";
--- a/toolkit/mozapps/extensions/internal/E10SAddonsRollout.jsm
+++ b/toolkit/mozapps/extensions/internal/E10SAddonsRollout.jsm
@@ -195,16 +195,18 @@ const RolloutPolicy = {
   "49limitedb": { addons: set49PaneOnly, webextensions: false },
 
   // Beta testing on 50
   "50allmpc": { addons: [], webextensions: true, mpc: true },
 
   "50a": { addons: set50Release, webextensions: true },
   "50b": { addons: set50Release, webextensions: false },
 
+  "51alladdons": { addons: [], webextensions: true, alladdons: true },
+
   "xpcshell-test": { addons: [ADDONS.test1, ADDONS.test2], webextensions: false },
 };
 
 Object.defineProperty(this, "isAddonPartOfE10SRollout", {
   configurable: false,
   enumerable: false,
   writable: false,
   value: function isAddonPartOfE10SRollout(aAddon) {
@@ -216,16 +218,24 @@ Object.defineProperty(this, "isAddonPart
     }
 
     if (blocklist && blocklist.indexOf(aAddon.id) > -1) {
       return false;
     }
 
     let policy = RolloutPolicy[policyId];
 
+    if (policy.alladdons) {
+      if (aAddon.multiprocessCompatible === false) {
+        return false;
+      }
+
+      return true;
+    }
+
     if (policy.webextensions && aAddon.type == "webextension") {
       return true;
     }
 
     if (policy.mpc && aAddon.multiprocessCompatible) {
       return true;
     }