Bug 1433735 - Cancel the TelemetryStopwatch when we stop monitoring a window for schedule pressure. r?mconley draft
authorJared Wein <jwein@mozilla.com>
Wed, 31 Jan 2018 11:08:35 -0500
changeset 749534 08c401957bc62ec55875cc642e4b6b32a931174e
parent 748898 08bf466172d3ac8b4819c09956012f8f67fd49ec
push id97414
push userbmo:jaws@mozilla.com
push dateWed, 31 Jan 2018 16:09:13 +0000
reviewersmconley
bugs1433735
milestone60.0a1
Bug 1433735 - Cancel the TelemetryStopwatch when we stop monitoring a window for schedule pressure. r?mconley MozReview-Commit-ID: HjlYc73Mzpa
browser/modules/SchedulePressure.jsm
--- a/browser/modules/SchedulePressure.jsm
+++ b/browser/modules/SchedulePressure.jsm
@@ -114,13 +114,15 @@ this.SchedulePressure = {
    */
   stopMonitoring(window) {
     function removeFromMapAndCancelTimeout(map, cancelFn) {
       if (map.has(window)) {
         cancelFn(map.get(window));
         map.delete(window);
       }
     }
+
+    TelemetryStopwatch.cancel("FX_SCHEDULE_PRESSURE_IDLE_SAMPLE_MS", window);
     removeFromMapAndCancelTimeout(this._setTimeoutWeakMap, window.clearTimeout);
     removeFromMapAndCancelTimeout(this._idleCallbackWeakMap, window.cancelIdleCallback);
     removeFromMapAndCancelTimeout(this._telemetryCallbackWeakMap, window.cancelIdleCallback);
   },
 };