Enable ESLint rule mozilla/use-services for tools/profiler

RESOLVED FIXED in Firefox 59

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
mozilla59
Points:
---

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment)

As we've been doing elsewhere, this continues the work to replace .getService() with Services.jsm uses where possible and enable the mozilla/use-services rule, this has small gains in performance (less boundary crossing) and generally more easy to read code.

For tools/profiler, this just affects test files.
Comment on attachment 8941766 [details]
Bug 1429705 - Enable ESLint rule mozilla/use-services for tools/profiler.

https://reviewboard.mozilla.org/r/212012/#review217772

Thanks!
Attachment #8941766 - Flags: review?(mstange) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a89326c8f25c
Enable ESLint rule mozilla/use-services for tools/profiler. r=mstange
https://hg.mozilla.org/mozilla-central/rev/a89326c8f25c
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.