1856567 - List nsIDirectoryServiceProvider[2] in components.conf for directory service

Status: NEW

(Core :: XPCOM, enhancement)

Description

[:Gijs (he/him)]

This would add it to the services JS linter and therefore make people use Services.dirsvc in more places.

I'm not aware of any other side effects but... we'll see? :-)

Comment 1

[:Gijs (he/him)]

To be clear, that's updating the interfaces bit at https://searchfox.org/mozilla-central/rev/37d9d6f0a77147292a87ab2d7f5906a62644f455/xpcom/io/components.conf#9-15 .

Comment 2

[Nika Layzell [:nika] (ni? for response)]

This seems reasonable and I don't think it'll have significant consequences. We'll just create the XPCWrappedJS with the extra methods from that new interface more eagerly when accessing it through the services getter. We might even want to expose nsIDirectoryServiceProvider2, if that's useful (though perhaps the interfaces could just be merged).