Closed Bug 1382192 Opened 2 years ago Closed 2 years ago

Use the prefs defined in TelemetryUtils.jsm in TelemetrySession.jsm

Categories

(Toolkit :: Telemetry, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: alejandro, Assigned: alejandro, Mentored)

References

(Blocks 1 open bug)

Details

(Whiteboard: [measurement:client])

Attachments

(1 file, 1 obsolete file)

We should change Telemetry code to use the preferences defined in the TelemetryUtils.jsm file rather than redefining them in TelemetrySession.jsm.
Whiteboard: [measurement:client]
Assignee: nobody → alexrs95
Attachment #8889440 - Attachment is obsolete: true
Attachment #8889440 - Flags: review?(alessio.placitelli)
Comment on attachment 8889446 [details] [diff] [review]
Use the prefs defined in TelemetryUtils.jsm in TelemetrySession.jsm

Review of attachment 8889446 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good, thanks!
Attachment #8889446 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/89ad2aed22193d69a31c16b26b274a00eec02a55
Bug 1382192 - Use the prefs defined in TelemetryUtils.jsm in TelemetrySession.jsm. r=dexter
https://hg.mozilla.org/mozilla-central/rev/89ad2aed2219
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.