Bug 1436423 - Reduce the schedule pressure limit closer to the values that users are reporting. r=mconley
authorJared Wein <jwein@mozilla.com>
Mon, 05 Mar 2018 14:09:39 -0500
changeset 406632 36c84344de23aa92f96c0d40076d5262aab32ad2
parent 406631 09978f543c018b2caea2cb132a2d55cc69b2647a
child 406633 e0a165295ef81483c10ae295769d6a5980b52225
child 406706 19838b896cd704a4cbf50d7e57b8d7dd8cafcbfd
push id60640
push userjwein@mozilla.com
push dateMon, 05 Mar 2018 22:26:44 +0000
treeherderautoland@36c84344de23 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmconley
bugs1436423
milestone60.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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(),