Bug 1324003 - Remove TELEMETRY_SCHEDULER_SEND_DAILY; r=chutten
authorVincent Lequertier <sky@riseup.net>
Mon, 19 Dec 2016 13:23:00 +0100
changeset 326798 c3745339764527e90df10cf435b024a570424c50
parent 326797 f425430a670307a7c8e595a98fa35920d2030a4f
child 326799 52c2c53e8340ec434da5ac3b86bbf6b3fd4be1d4
push id31111
push usercbook@mozilla.com
push dateWed, 21 Dec 2016 15:57:24 +0000
treeherdermozilla-central@10a5e2fc24e4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerschutten
bugs1324003
milestone53.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 1324003 - Remove TELEMETRY_SCHEDULER_SEND_DAILY; r=chutten
toolkit/components/telemetry/Histograms.json
toolkit/components/telemetry/TelemetrySession.jsm
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -5994,23 +5994,16 @@
   },
   "TELEMETRY_SCHEDULER_WAKEUP": {
     "alert_emails": ["telemetry-client-dev@mozilla.com"],
     "bug_numbers": [1250640],
     "expires_in_version": "53",
     "kind": "count",
     "description": "Count of TelemetrySession scheduler ticks that were delayed long enough to suspect sleep."
   },
-  "TELEMETRY_SCHEDULER_SEND_DAILY": {
-    "alert_emails": ["telemetry-client-dev@mozilla.com"],
-    "bug_numbers": [1250640],
-    "expires_in_version": "53",
-    "kind": "count",
-    "description": "Count of TelemetrySession triggering a daily ping."
-  },
   "TELEMETRY_TEST_FLAG": {
     "alert_emails": ["telemetry-client-dev@mozilla.com"],
     "expires_in_version": "never",
     "kind": "flag",
     "description": "a testing histogram; not meant to be touched"
   },
   "TELEMETRY_TEST_COUNT": {
     "alert_emails": ["telemetry-client-dev@mozilla.com"],
--- a/toolkit/components/telemetry/TelemetrySession.jsm
+++ b/toolkit/components/telemetry/TelemetrySession.jsm
@@ -472,17 +472,16 @@ var TelemetryScheduler = {
       this._log.trace("_schedulerTickLogic - First scheduler tick after sleep.");
     }
     this._lastTickTime = now;
 
     // Check if the daily ping is due.
     const shouldSendDaily = this._isDailyPingDue(nowDate);
 
     if (shouldSendDaily) {
-      Telemetry.getHistogramById("TELEMETRY_SCHEDULER_SEND_DAILY").add(1);
       this._log.trace("_schedulerTickLogic - Daily ping due.");
       this._lastDailyPingTime = now;
       return Impl._sendDailyPing();
     }
 
     // Check if the aborted-session ping is due. If a daily ping was saved above, it was
     // already duplicated as an aborted-session ping.
     const isAbortedPingDue =