use Services.jsm more in specialpowersAPI.js

RESOLVED FIXED in mozilla32

Status

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: froydnj, Unassigned)

Tracking

unspecified
mozilla32
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

No description provided.
There's no reason we should be groveling around in XPCOM when Services.jsm has
all these services already.  Might even be a bit faster, too.
Attachment #8414611 - Flags: review?(jmaher)
Comment on attachment 8414611 [details] [diff] [review]
use Services.jsm more in specialpowersAPI.js

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

I admit my services.jsm usage is very low, from what I can tell this looks good and assuming these are identical methods this looks good. Please test on try, including mobile platforms.
Attachment #8414611 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/mozilla-central/rev/ab6426d96c6d
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.