1448798 - Change devtools/shared/telemetry.js to an ES6 class

Status: RESOLVED FIXED

(DevTools :: General, enhancement, P2)

Description

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

A lot of work has gone into optimizing ES6 classes and upgrading is simple so we should make the switch and save ourselves a few ms here and there.

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Attachment: Bug 1448798 - Change devtools/shared/telemetry.js to an ES6 class

Review commit: https://reviewboard.mozilla.org/r/231146/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/231146/

Comment 2

[Patrick Brosset <:pbro>]

Comment on attachment 8962284 [details]
Bug 1448798 - Change devtools/shared/telemetry.js to an ES6 class

https://reviewboard.mozilla.org/r/231146/#review236566

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/459de2510722
Change devtools/shared/telemetry.js to an ES6 class r=pbro

Comment 4

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/459de2510722

Comment 5

[:Harald Kirschner :digitarald]

Moving to M1 for splitting off M2 work.