Bug 748520 - persist failed telemetry pings for possible re-sending later; r=vdjeric
authorNathan Froyd <froydnj@mozilla.com>
Mon, 10 Dec 2012 17:09:40 -0500
changeset 115962 6c224e8173903be63d198fd55d9e83f6f1b0ae4e
parent 115961 cc544979aeaa363bad3b6a8641a33b9330c24078
child 115963 90b2c0cb4a91da8527f5ca701f53eeb5bdc6919a
push id24034
push useremorley@mozilla.com
push dateFri, 14 Dec 2012 15:28:57 +0000
treeherdermozilla-central@50d8f411d305 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvdjeric
bugs748520
milestone20.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 748520 - persist failed telemetry pings for possible re-sending later; r=vdjeric
toolkit/components/telemetry/TelemetryPing.js
--- a/toolkit/components/telemetry/TelemetryPing.js
+++ b/toolkit/components/telemetry/TelemetryPing.js
@@ -609,16 +609,17 @@ TelemetryPing.prototype = {
     } catch (e if e instanceof StopIteration) {
       finishPings(reason);
       return;
     }
     function onSuccess() {
       this.sendPingsFromIterator(server, reason, i);
     }
     function onError() {
+      this.savePing(data, true);
       // Notify that testing is complete, even if we didn't send everything.
       finishPings(reason);
     }
     this.doPing(server, data,
                 onSuccess.bind(this), onError.bind(this));
   },
 
   finishPingRequest: function finishPingRequest(success, startTime, ping) {