Bug 1436423 - Reduce the schedule pressure limit closer to the values that users are reporting. r?mconley draft
authorJared Wein <jwein@mozilla.com>
Mon, 05 Mar 2018 14:09:39 -0500
changeset 763328 ca884bf12af3c843d08b98e3432c8231b5596b92
parent 763326 e88148e0e4e1e8dd72f2acf6c535c38fc1c2a7c1
push id101403
push userbmo:jaws@mozilla.com
push dateMon, 05 Mar 2018 19:11:58 +0000
reviewersmconley
bugs1436423
milestone60.0a1
Bug 1436423 - Reduce the schedule pressure limit closer to the values that users are reporting. r?mconley Note that this value, 300, is still far above the 95% threshold that telemetry is reporting (59 milliseconds) so this won't be noticeable to most users. The > 1% of users who are having this issue will benefit greatly from this change. MozReview-Commit-ID: Bd51gjc5z83
browser/app/profile/firefox.js
browser/modules/SchedulePressure.jsm
--- a/browser/app/profile/firefox.js
+++ b/browser/app/profile/firefox.js
@@ -227,17 +227,17 @@ pref("general.smoothScroll", true);
 pref("general.autoScroll", false);
 #else
 pref("general.autoScroll", true);
 #endif
 
 pref("browser.stopReloadAnimation.enabled", true);
 pref("browser.schedulePressure.enabled", true);
 pref("browser.schedulePressure.defaultCount", 3);
-pref("browser.schedulePressure.timeoutMs", 1000);
+pref("browser.schedulePressure.timeoutMs", 300);
 
 // UI density of the browser chrome. This mostly affects toolbarbutton
 // and urlbar spacing. The possible values are 0=normal, 1=compact, 2=touch.
 pref("browser.uidensity", 0);
 // Whether Firefox will automatically override the uidensity to "touch"
 // while the user is in a touch environment (such as Windows tablet mode).
 pref("browser.touchmode.auto", true);
 
--- a/browser/modules/SchedulePressure.jsm
+++ b/browser/modules/SchedulePressure.jsm
@@ -7,17 +7,17 @@
 var EXPORTED_SYMBOLS = ["SchedulePressure"];
 
 ChromeUtils.import("resource://gre/modules/XPCOMUtils.jsm");
 ChromeUtils.defineModuleGetter(this, "TelemetryStopwatch",
   "resource://gre/modules/TelemetryStopwatch.jsm");
 XPCOMUtils.defineLazyPreferenceGetter(this, "SCHEDULE_PRESSURE_ENABLED",
   "browser.schedulePressure.enabled", true);
 XPCOMUtils.defineLazyPreferenceGetter(this, "TIMEOUT_AMOUNT",
-  "browser.schedulePressure.timeoutMs", 1000);
+  "browser.schedulePressure.timeoutMs", 300);
 
 /**
  * The SchedulePressure object provides the ability to alter
  * the behavior of a program based on the idle activity of the
  * host machine.
  */
 var SchedulePressure = {
   _idleCallbackWeakMap: new WeakMap(),