Closed Bug 1266832 Opened 4 years ago Closed 4 years ago

replace Services.telemetry

Categories

(DevTools :: Framework, enhancement, P1)

enhancement

Tracking

(firefox50 fixed)

RESOLVED FIXED
Firefox 50
Iteration:
50.4 - Aug 1
Tracking Status
firefox50 --- fixed

People

(Reporter: tromey, Assigned: tromey)

References

Details

(Whiteboard: [devtools-html])

Attachments

(1 file)

We need to replace Services.telemetry for the devtools
de-chrome-ification project.
Flags: qe-verify-
Priority: -- → P2
Priority: P2 → P3
Whiteboard: [devtools-html] → [reserve-html]
Specifically our plan for this bug is to stub out telemetry and have it do nothing.
Probably the best way is to make a Services.telemetry object in our Services.js shim.
For some reason this one was making me crazy.
Assignee: nobody → ttromey
Status: NEW → ASSIGNED
Iteration: --- → 50.4 - Aug 1
Priority: P3 → P1
Whiteboard: [reserve-html] → [devtools-html]
Comment on attachment 8772998 [details]
Bug 1266832 - add dummy telemetry object to Services shim;

https://reviewboard.mozilla.org/r/65650/#review62914

LGTM.
Attachment #8772998 - Flags: review?(gtatum) → review+
Pushed by ttromey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b92567e80d1e
add dummy telemetry object to Services shim; r=gregtatum
https://hg.mozilla.org/mozilla-central/rev/b92567e80d1e
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.