[tor-commits] [tor-browser/tor-browser-52.0.2esr-7.0-2] Bug 21876: Always use esr policies for e10s.

[gk at torproject.org]

commit 1240ac3e85536322d6af0075c662eaf60ab078f9
Author: Kathy Brade <brade at pearlcrescent.com>
Date:   Tue Apr 11 11:33:02 2017 -0400

    Bug 21876: Always use esr policies for e10s.
    
    Always use the policies associated with the esr update channel so that
    the e10s behavior is the same for all Tor Browser builds.
---
 browser/extensions/e10srollout/bootstrap.js | 5 ++++-
 toolkit/xre/nsAppRunner.cpp                 | 9 +++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/browser/extensions/e10srollout/bootstrap.js b/browser/extensions/e10srollout/bootstrap.js
index 8847501..e4e0738 100644
--- a/browser/extensions/e10srollout/bootstrap.js
+++ b/browser/extensions/e10srollout/bootstrap.js
@@ -58,7 +58,10 @@ function defineCohort() {
   }
   cohortDefinedOnThisSession = true;
 
-  let updateChannel = UpdateUtils.getUpdateChannel(false);
+  // For Tor Browser, always use the e10s policies associated with the esr
+  // update channel so that the e10s behavior is the same for all builds.
+  let updateChannel = "esr";
+
   if (!(updateChannel in TEST_THRESHOLD)) {
     setCohort("unsupportedChannel");
     return;
diff --git a/toolkit/xre/nsAppRunner.cpp b/toolkit/xre/nsAppRunner.cpp
index 721f599..190fec1 100644
--- a/toolkit/xre/nsAppRunner.cpp
+++ b/toolkit/xre/nsAppRunner.cpp
@@ -5453,15 +5453,12 @@ MultiprocessBlockPolicy() {
 #endif
 
   /**
-   * Avoids enabling e10s for Windows XP users on the release channel.
+   * Avoid enabling e10s for all Windows XP users.
    */
 #if defined(XP_WIN)
   if (!IsVistaOrLater()) {
-    nsAdoptingCString channelName = Preferences::GetDefaultCString("app.update.channel");
-    if (channelName.EqualsLiteral("release") || channelName.EqualsLiteral("esr")) {
-      gMultiprocessBlockPolicy = kE10sDisabledForOperatingSystem;
-      return gMultiprocessBlockPolicy;
-    }
+    gMultiprocessBlockPolicy = kE10sDisabledForOperatingSystem;
+    return gMultiprocessBlockPolicy;
   }
 #endif // XP_WIN