Benjamin Smedberg Reporter
	Description • 9 years ago

In both TelemetryPing and TelemetrySession there are 

const PREF_FHR_UPLOAD_ENABLED = "datareporting.healthreport.uploadEnabled";

These prefs are never used. I'm pretty sure the bug is that this prefs should be used in TelemetryPing: this pref represents the checkbox in prefs for disabling sending FHR and telemetry.

	Benjamin Smedberg Reporter
	Comment 1 • 9 years ago

This will also fix problems where xpcshell tests are calling TelemetryPing.send and crashing: the uploadEnabled pref should be missing/false in the normal xpcshell environment.

	Georg Fritzsche [:gfritzsche]
	Comment 2 • 9 years ago

Alessio, is that something that will get cleared up by bug 1134279?

	Alessio Placitelli [:Dexter]
	Comment 3 • 9 years ago

That's something being done in bug 1137252 (|_canSend|).

	Georg Fritzsche [:gfritzsche]
	Comment 4 • 9 years ago

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #1)
> This will also fix problems where xpcshell tests are calling
> TelemetryPing.send and crashing: the uploadEnabled pref should be
> missing/false in the normal xpcshell environment.

What is the crash here? Is it for outbound network traffic in tests?

If so, we should just set this to a proper dummy URL in test profiles that doesn't trigger this.

	Georg Fritzsche [:gfritzsche]
	Comment 5 • 9 years ago

(In reply to Georg Fritzsche [:gfritzsche] from comment #4)
> (In reply to Benjamin Smedberg  [:bsmedberg] from comment #1)
> > This will also fix problems where xpcshell tests are calling
> > TelemetryPing.send and crashing: the uploadEnabled pref should be
> > missing/false in the normal xpcshell environment.
> 
> What is the crash here? Is it for outbound network traffic in tests?
> 
> If so, we should just set this to a proper dummy URL in test profiles that
> doesn't trigger this.

... which bug 1137252 apparently takes care off too.

	Georg Fritzsche [:gfritzsche]
	Comment 6 • 9 years ago

(In reply to Georg Fritzsche [:gfritzsche] from comment #5)
> (In reply to Georg Fritzsche [:gfritzsche] from comment #4)
> > (In reply to Benjamin Smedberg  [:bsmedberg] from comment #1)
> > > This will also fix problems where xpcshell tests are calling
> > > TelemetryPing.send and crashing: the uploadEnabled pref should be
> > > missing/false in the normal xpcshell environment.
> > 
> > What is the crash here? Is it for outbound network traffic in tests?
> > 
> > If so, we should just set this to a proper dummy URL in test profiles that
> > doesn't trigger this.
> 
> ... which bug 1137252 apparently takes care off too.

Bug 1137252 didn't actually handle this for xpcshell, but i ran into this in bug 1150134 and am fixing it there.