580676 - Add content prefs service service to Services.jsm

Status: RESOLVED FIXED

(Core :: General, enhancement)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch v0.1

nsIContentPrefService could be added to Services.jsm

Comment 1

[Dave Garrett]

Shouldn't you use defineLazyServiceGetter instead of defineLazyGetter here?

Comment 2

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 459057 [details] [diff] [review]
Patch v0.1

a=beltzner

Comment 3

[:Gavin Sharp [email: [e-mail]]]

(In reply to comment #1)
> Shouldn't you use defineLazyServiceGetter instead of defineLazyGetter here?

Oh, I guess so. Should be grouped with those too. Vivien: can you make that change when checking in?

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #3)
> (In reply to comment #1)
> > Shouldn't you use defineLazyServiceGetter instead of defineLazyGetter here?
> 
> Oh, I guess so. Should be grouped with those too. Vivien: can you make that
> change when checking in?

Sure, I will do.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

http://hg.mozilla.org/mozilla-central/rev/3ccadf603301