Closed
Bug 1150975
Opened 10 years ago
Closed 10 years ago
Telemetry should not know about browser.selfsupport.enabled
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
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)
|
||
Comment 1•10 years ago
|
||
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)
|
||
Comment 2•10 years ago
|
||
Handled per comment 1.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•