Bug 1435067 - Use the pingsender when sending the sync ping r=gfritzsche,markh
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Thu, 01 Feb 2018 16:38:41 -0500
changeset 406084 3aa4c86bbf163e222d9789f9f43912e7d24b3763
parent 406083 885ab31f399bc91eeb70d4721e2c6c90e56508fc
child 406085 017d83ffe0a47a22d342ab61224ca09977a308a5
push id60459
push usertchiovoloni@mozilla.com
push dateThu, 01 Mar 2018 21:12:36 +0000
treeherderautoland@3aa4c86bbf16 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgfritzsche, markh
bugs1435067
milestone60.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 1435067 - Use the pingsender when sending the sync ping r=gfritzsche,markh MozReview-Commit-ID: 2x98suZvfr0
services/sync/modules/telemetry.js
--- a/services/sync/modules/telemetry.js
+++ b/services/sync/modules/telemetry.js
@@ -495,17 +495,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) {