Closed Bug 1003296 Opened 10 years ago Closed 10 years ago

use Services.jsm more in specialpowersAPI.js

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla32

People

(Reporter: froydnj, Unassigned)

Details

Attachments

(1 file)

      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: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.