Open Bug 1617896 Opened 5 years ago Updated 2 years ago

NetworkGeolocationProvider can be accessed as an instance or a service

Categories

(Core :: DOM: Geolocation, defect, P2)

defect

Tracking

()

People

(Reporter: standard8, Unassigned)

Details

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.

No longer blocks: search-optimization

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

No longer blocks: search-region
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.