Bug 1435067 - Use the pingsender when sending the sync ping r?markh draft
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Thu, 01 Feb 2018 16:38:41 -0500
changeset 750304 67ebf60f1d6691c04cc60a4fd607269f943a9bc0
parent 749604 3804441e575c9f46fcb03894de3c780eeae7197f
push id97613
push userbmo:tchiovoloni@mozilla.com
push dateThu, 01 Feb 2018 21:38:58 +0000
reviewersmarkh
bugs1435067
milestone60.0a1
Bug 1435067 - Use the pingsender when sending the sync ping r?markh MozReview-Commit-ID: 2x98suZvfr0
services/sync/modules/telemetry.js
--- a/services/sync/modules/telemetry.js
+++ b/services/sync/modules/telemetry.js
@@ -491,17 +491,17 @@ class SyncTelemetryImpl {
       return false;
     }
     // We still call submit() with possibly illegal payloads so that tests can
     // know that the ping was built. We don't end up submitting them, however.
     let numEvents = record.events ? record.events.length : 0;
     if (record.syncs.length || numEvents) {
       log.trace(`submitting ${record.syncs.length} sync record(s) and ` +
                 `${numEvents} event(s) to telemetry`);
-      TelemetryController.submitExternalPing("sync", record);
+      TelemetryController.submitExternalPing("sync", record, { usePingSender: true });
       return true;
     }
     return false;
   }
 
   onSyncStarted(data) {
     const why = data && JSON.parse(data).why;
     if (this.current) {