Closed
Bug 1263804
Opened 8 years ago
Closed 8 years ago
Remove last references to Metrics.jsm
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: Unfocused, Assigned: Unfocused)
References
Details
Attachments
(1 file)
2.53 KB,
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
Bug 1235345 removed services/metrics, but seems to have missed a couple of references to Metrics.jsm - which didn't break because they're in unused lazy getters.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8740238 -
Flags: review?(gijskruitbosch+bugs)
Comment 2•8 years ago
|
||
Comment on attachment 8740238 [details] [diff] [review] Patch v1 Review of attachment 8740238 [details] [diff] [review]: ----------------------------------------------------------------- r=me Should https://dxr.mozilla.org/mozilla-central/source/toolkit/components/telemetry/docs/fhr/architecture.rst#11 get updated as well?
Attachment #8740238 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 3•8 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #2) > Should > https://dxr.mozilla.org/mozilla-central/source/toolkit/components/telemetry/ > docs/fhr/architecture.rst#11 get updated as well? I'm going to assume not - it's not just that line, but that entire directory that needs removed (since FHR itself was removed in bug 1234526). index.rst in that directory includes the following: > **Firefox Health Report (FHR) is obsolete and no longer ships with Firefox. > This documentation will live here for a few more cycles.**
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/543033638027
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in
before you can comment on or make changes to this bug.
Description
•