NetworkGeolocationProvider is currently accessed in various places by creating it as an instance, and as a service.

I believe that the service case is so that tests can override the factory, and replace the built-in component with a test version.

From what I can tell, we only ever create the instances once.

Bug 1589618 is creating another access of it (via js), but we're doing that in the knowledge that we should review the architecture of that component with respect to the future work, and act on this soon.

So part of this is going to be fixed by https://bugzilla.mozilla.org/show_bug.cgi?id=1627538 that will remove the 3rd JS access of this module, leaving open for owners to decide about the instance vs service part