Closed Bug 1217448 Opened 9 years ago Closed 8 years ago

Move the sessionRecorder initialization after |enableTelemetryRecording|

Categories

(Toolkit :: Telemetry, defect, P4)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1234526
Tracking Status
firefox44 --- affected

People

(Reporter: Dexter, Unassigned)

Details

(Whiteboard: [measurement:client] [wait until fhr is disabled])

In TelemetryController.jsm, we init the SessionRecorder before checking if UT is enabled [1].

We should move this after the |enableTelemetryRecording| block and drop the PREF_FHR_ENABLED check once we permanently switch over to unified Telemetry.

[1] - https://dxr.mozilla.org/mozilla-central/source/toolkit/components/telemetry/TelemetryController.jsm#725
Priority: -- → P4
Whiteboard: [measurement:client]
Whiteboard: [measurement:client] → [measurement:client] [wait until fhr is disabled]
Blocks: 1217452
No longer blocks: 1217452
Blocks: 1217457
Blocks: 1217458
No longer blocks: 1217458
No longer blocks: 1217457
Alessio, i think you did that in another bug already - can you dupe this?
Flags: needinfo?(alessio.placitelli)
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(alessio.placitelli)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.