Closed Bug 1150975 Opened 9 years ago Closed 9 years ago

Telemetry should not know about browser.selfsupport.enabled

Categories

(Toolkit :: Telemetry, defect)

Product:
Toolkit
Component:
Telemetry
Platform:
x86
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: benjamin, Unassigned)

References

Details

Currently there is in TelemetryPing.jsm a pref-check on browser.selfsupport.enabled. There is no reason for this pref check and it couples self-support and telemetry in a weird way.

I'm pretty sure it should be removed, but I'm confused about why it was added at all. It was added by Alessio in bug 1134279
Flags: needinfo?(alessio.placitelli)
I'm aware of this issue, the confusion started in bug 1134279 comment 11 (a base probe should be allowed to record if FHR is ON or if self-support is ON) but you already cleared it up in bug 1137252 comment 13.

Bug 1137252 fixes this issue by removing the self-support pref check, but I can fix the issue here if you prefer!
Flags: needinfo?(alessio.placitelli)
Handled per comment 1.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.