578122 - Use Services.jsm in Fennec

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: patch

This patch updates Fennec to use the lazy getters from Services.jsm <http://www.gavinsharp.com/blog/2010/02/25/services-jsm/> where appropriate.  This allows us to remove ~170 LOC, and should reduce service instantiation overhead in a few places where we weren't using our own lazy getters.

Comment 1

[Matt Brubeck (:mbrubeck)]

Comment on attachment 456898 [details] [diff] [review]
patch

There are some errors caused by this, new version coming.

Comment 2

[Matt Brubeck (:mbrubeck)]

Attachment: patch

Fix warning in sanitize.js and error in config.js.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 456964 [details] [diff] [review]
patch

>diff -r 4e38010bf28a chrome/content/sanitize.js

>   sanitize: function ()
>   {
>-    var psvc = Components.classes["@mozilla.org/preferences-service;1"]
>-                         .getService(Components.interfaces.nsIPrefService);
>+    var psvc = Cc["@mozilla.org/preferences-service;1"].getService(Ci.nsIPrefService);
>     var branch = psvc.getBranch(this._prefDomain);

Change to | var branch = Services.prefs.getBranch(this._prefDomain); |
and remove the line above it

>     geolocation: {

>         // clear any network geolocation provider sessions
>-        var psvc = Components.classes["@mozilla.org/preferences-service;1"]
>-                             .getService(Components.interfaces.nsIPrefService);
>+        var psvc = Cc["@mozilla.org/preferences-service;1"]
>+                     .getService(Components.interfaces.nsIPrefService);
>         try {
>           var branch = psvc.getBranch("geo.wifi.access_token.");
>           branch.deleteBranch("");
>           
>           branch = psvc.getBranch("geo.request.remember.");
>           branch.deleteBranch("");

Same...

r+ with those changes. I'll make the changes before landing

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/23a684e04341