Bug 791765 - make sure to try and send all the ping files; r=taras
authorNathan Froyd <froydnj@mozilla.com>
Mon, 01 Oct 2012 12:52:00 -0400
changeset 115065 dc17abff955a4ac9647996c85b0c57b2b13ac413
parent 115064 85f3e07f71cee5d291ba130399c1a7cb7740213a
child 115066 48724698ca340cd4c2ed22d39f106f6e51ff7a8c
push id1708
push userakeybl@mozilla.com
push dateMon, 19 Nov 2012 21:10:21 +0000
treeherdermozilla-beta@27b14fe50103 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstaras
bugs791765
milestone18.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 791765 - make sure to try and send all the ping files; r=taras
toolkit/components/telemetry/TelemetryPing.js
--- a/toolkit/components/telemetry/TelemetryPing.js
+++ b/toolkit/components/telemetry/TelemetryPing.js
@@ -508,17 +508,17 @@ TelemetryPing.prototype = {
     payloadObj.info = this.getMetadata(reason);
     return { slug: slug, payload: JSON.stringify(payloadObj) };
   },
 
   getPayloads: function getPayloads(reason) {
     function payloadIter() {
       yield this.getCurrentSessionPayloadAndSlug(reason);
 
-      if (this._pendingPings.length > 0) {
+      while (this._pendingPings.length > 0) {
         let data = this._pendingPings.pop();
         // Send persisted pings to the test URL too.
         if (reason == "test-ping") {
           data.slug = reason;
         }
         yield data;
       }
     }