Bug 1336598 P1 Tune setTimeout anti-flood constants to minimize jank. r=smaug a=lizzard
authorBen Kelly <ben@wanderview.com>
Mon, 13 Feb 2017 15:02:30 -0500
changeset 378422 c234f1f1ed00994e018b49bfabedd104dbfa173a
parent 378421 a453af7168f3290b05e01aca27dbda196d5da62a
child 378423 a07f4b1cef63068d31e8ea17130d5068cc8e05a3
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssmaug, lizzard
Bug 1336598 P1 Tune setTimeout anti-flood constants to minimize jank. r=smaug a=lizzard
--- a/dom/base/TimeoutManager.cpp
+++ b/dom/base/TimeoutManager.cpp
@@ -64,22 +64,22 @@ namespace {
 // The number of queued runnables within the TabGroup ThrottledEventQueue
 // at which to begin applying back pressure to the window.
 const uint32_t kThrottledEventQueueBackPressure = 5000;
 // The amount of delay to apply to timers when back pressure is triggered.
 // As the length of the ThrottledEventQueue grows delay is increased.  The
 // delay is scaled such that every kThrottledEventQueueBackPressure runnables
 // in the queue equates to an additional kBackPressureDelayMS.
-const double kBackPressureDelayMS = 500;
+const double kBackPressureDelayMS = 250;
 // This defines a limit for how much the delay must drop before we actually
 // reduce back pressure throttle amount.  This makes the throttle delay
 // a bit "sticky" once we enter back pressure.
-const double kBackPressureDelayReductionThresholdMS = 400;
+const double kBackPressureDelayReductionThresholdMS = 1000;
 // The minimum delay we can reduce back pressure to before we just floor
 // the value back to zero.  This allows us to ensure that we can exit
 // back pressure event if there are always a small number of runnables
 // queued up.
 const double kBackPressureDelayMinimumMS = 100;
 // Convert a ThrottledEventQueue length to a timer delay in milliseconds.