Remove last references to Metrics.jsm

RESOLVED FIXED in Firefox 48

Status

()

Firefox
General
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Unfocused, Assigned: Unfocused)

Tracking

unspecified
Firefox 48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

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: nobody → bmcbride
Status: NEW → ASSIGNED
Created attachment 8740238 [details] [diff] [review]
Patch v1
Attachment #8740238 - Flags: review?(gijskruitbosch+bugs)

Comment 2

2 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+
(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

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/543033638027
Status: ASSIGNED → RESOLVED
Last Resolved: 2 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.