Closed Bug 1381843 Opened 7 years ago Closed 7 years ago

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

Categories

(Toolkit :: Telemetry, enhancement, P1)

Product:
Toolkit
Component:
Telemetry
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox56 --- fixed

People

(Reporter: alejandro, Assigned: alejandro, Mentored)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Use the prefs defined in TelemetryUtils.jsm in TelemetrySend.jsm
4.74 KB, patch
Dexter
: review+
Details | Diff | Splinter Review
We should change Telemetry code to use the preferences defined in the TelemetryUtils.jsm file rather than redefining them in TelemetrySend.jsm.
Priority: P3 → P1
Assignee: nobody → alexrs95
Comment on attachment 8887844 [details] [diff] [review] Use the prefs defined in TelemetryUtils.jsm in TelemetrySend.jsm Review of attachment 8887844 [details] [diff] [review]: ----------------------------------------------------------------- This looks good, thanks!
Attachment #8887844 - Flags: review?(alessio.placitelli) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/4194f4a318defa521543768acf09e19efba899ac Bug 1381843 - Use the prefs defined in TelemetryUtils.jsm in TelemetrySend.jsm. r=dexter
https://hg.mozilla.org/mozilla-central/rev/4194f4a318de
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: