Closed Bug 1209088 Opened 4 years ago Closed 4 years ago

[meta] Disable & remove FHR on desktop

Categories

(Firefox Health Report Graveyard :: Client: Desktop, defect, P3)

defect

Tracking

(firefox44 affected)

RESOLVED FIXED
Tracking Status
firefox44 --- affected

People

(Reporter: gfritzsche, Unassigned)

References

Details

(Keywords: meta, Whiteboard: [measurement:client] [measurement:client:project])

Once we landed bug 1208376 (disabling FHR upload), we can start to investigate fully disabling FHR on desktop and let it ride the trains.

If that goes well we can later remove the FHR desktop code.
Let's not move too slowly. I think we can remove the code immediately.
Priority: P3 → P2
Points: --- → 2
Points: 2 → ---
Priority: P2 → P3
Depends on: 1210815
Keywords: meta
Summary: Disable FHR on desktop → [meta] Disable & remove FHR on desktop
Depends on: 1230530
Removing the providers in bug 1230530 should be a rather stand-alone bug to work on.

Other work i can think of:
* removal of all of services/datareporting, services/healthreport, services/metrics
* investigating code covered by MOZ_SERVICES_HEALTHREPORT, MOZ_SERVICES_METRICS and any AppConstants.jsm equivalents:
  https://dxr.mozilla.org/mozilla-central/search?q=MOZ_SERVICES_HEALTHREPORT&redirect=false&case=true
  https://dxr.mozilla.org/mozilla-central/search?q=MOZ_SERVICES_METRICS&redirect=false&case=true
* find out if we have other build flags here
* investigate code that uses "@mozilla.org/datareporting/service;1" et al
* watch out that we don't have any JS/JSMs left in any package-manifest.in or other registrations
* remove any now-unused preferences for metrics, healthreport, datareporting

One problem is that we have to coordinate properly with the Android side (bug 1230206 and bug 1183320) for clean landings and not introducing temporary workarounds etc.
Depends on: 1230206
Whiteboard: [unifiedTelemetry] [measurement:client] → [measurement:client] [measurement:client:tracking]
The Android codebase is entirely separate, and we don't ship any of the desktop code. 

We do use MOZ_SERVICES_HEALTHREPORT, but it's set in mobile/android/confvars.sh, so whatever you do to the desktop confvars shouldn't break anything. We use that to enable about:healthreport, and AIUI we're keeping that around, so perhaps it'll stay set.
See Also: → 1183320
Depends on: 1234518
Depends on: 1234522
Depends on: 1234526
Depends on: 1235345
Depends on: 1236551
Depends on: 1236588
Whiteboard: [measurement:client] [measurement:client:tracking] → [measurement:client] [measurement:client:tracking] [measurement:client:project]
Whiteboard: [measurement:client] [measurement:client:tracking] [measurement:client:project] → [measurement:client] [measurement:client:project]
Depends on: 1237978
Depends on: 1237983
Closing this, all the dependencies landed.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in before you can comment on or make changes to this bug.