Bug 1401685 - Don't skip sending BHR ping if it is empty, r=froydnj
authorMichael Layzell <michael@thelayzells.com>
Wed, 20 Sep 2017 15:32:53 -0400
changeset 431557 06eacffae8c8d2bec170b2ad6e6bfa4f95410e39
parent 431553 789e21a08030b00acd832a58268680dede2e7a1b
child 431558 a88ee0137c104a57c747cd129c45952ccf1431ad
push id7785
push userryanvm@gmail.com
push dateThu, 21 Sep 2017 13:39:55 +0000
treeherdermozilla-beta@06d4034a8a03 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj
bugs1401685
milestone57.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 1401685 - Don't skip sending BHR ping if it is empty, r=froydnj
toolkit/components/backgroundhangmonitor/BHRTelemetryService.js
--- a/toolkit/components/backgroundhangmonitor/BHRTelemetryService.js
+++ b/toolkit/components/backgroundhangmonitor/BHRTelemetryService.js
@@ -86,18 +86,17 @@ BHRTelemetryService.prototype = Object.f
     if (!Services.telemetry.canRecordExtended) {
       return;
     }
 
     // NOTE: We check a separate bhrPing.enabled pref here. This pref is unset
     // when running tests so that we run as much of BHR as possible (to catch
     // errors) while avoiding timeouts caused by invoking `pingsender` during
     // testing.
-    if (Services.prefs.getBoolPref("toolkit.telemetry.bhrPing.enabled", false) &&
-        this.payload.hangs.length > 0) {
+    if (Services.prefs.getBoolPref("toolkit.telemetry.bhrPing.enabled", false)) {
       this.payload.timeSinceLastPing = new Date() - this.startTime;
       TelemetryController.submitExternalPing("bhr", this.payload, {
         addEnvironment: true,
       });
     }
     this.resetPayload();
   },