Closed
Bug 1421547
Opened 7 years ago
Closed 7 years ago
Remove remaining traces of the old toolkit.telemetry.enabledPreRelease pref
Categories
(Core :: Preferences: Backend, enhancement)
Core
Preferences: Backend
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
Attachments
(1 file)
1.91 KB,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
The code to migrate from the toolkit.telemetry.enabledPreRelease pref to
toolkit.telemetry.enabled was added to Firefox 31 in bug 986582. It should be
safe to remove now.
MozReview-Commit-ID: JBkn20bUQXx
Assignee | ||
Comment 1•7 years ago
|
||
rnewman, despite what the commit message says, I'm not 100% certain this is
safe to remove. I figure you'll probably know!
Attachment #8932767 -
Flags: review?(rnewman)
Comment 2•7 years ago
|
||
Comment on attachment 8932767 [details] [diff] [review]
Remove remaining traces of the old toolkit.telemetry.enabledPreRelease pref
Fine by me!
CCing liuche so that this passes before the eyes of a data steward.
Flags: needinfo?(liuche)
Attachment #8932767 -
Flags: review?(rnewman) → review+
Comment 3•7 years ago
|
||
Removing data probe, looks fine to me! Is there any documentation that needs to be updated?
Flags: needinfo?(liuche)
Assignee | ||
Comment 4•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/197515c59832ddadf946ef99ea1c4353735413a9
Bug 1421547 - Remove remaining traces of the old toolkit.telemetry.enabledPreRelease pref. r=rnewman
Pushed by nnethercote@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/197515c59832
Remove remaining traces of the old toolkit.telemetry.enabledPreRelease pref. r=rnewman
Comment 6•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee | ||
Comment 7•7 years ago
|
||
I am not aware of any documentation that needs updating, but I could easily be overlooking something.
You need to log in
before you can comment on or make changes to this bug.
Description
•