Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Bug 762590 - don't send test pings at idle when !Telemetry.canSend; r…

…=taras,a=akeybl
  • Loading branch information...
commit 103c62b6a247232d1f24f362d1fa5efacd9d8e14 1 parent 002d014
@froydnj froydnj authored
Showing with 6 additions and 4 deletions.
  1. +6 −4 toolkit/components/telemetry/TelemetryPing.js
View
10 toolkit/components/telemetry/TelemetryPing.js
@@ -744,10 +744,12 @@ TelemetryPing.prototype = {
idleService.removeIdleObserver(this, IDLE_TIMEOUT_SECONDS);
this._isIdleObserver = false;
}
- reason = (Telemetry.canSend && aTopic == "idle"
- ? "idle-daily"
- : "test-ping");
- this.send(reason, server);
+ if (aTopic == "test-ping") {
+ this.send("test-ping", server);
+ }
+ else if (Telemetry.canSend && aTopic == "idle") {
+ this.send("idle-daily", server);
+ }
break;
case "quit-application-granted":
Telemetry.saveHistograms(this.savedHistogramsFile(),
Please sign in to comment.
Something went wrong with that request. Please try again.