2022043 - Permissions API for geolocation should reflect Windows OS location permission

Status: RESOLVED FIXED

(Toolkit :: Geolocation Provider, enhancement)

Description

[Haik Aftandilian [:haik]]

On Windows, when Firefox access to the OS location is denied, the permissions API should reflect that instead of having our location system fall back to using GLS. This is the Windows version of bug 1529591.

Comment 1

[Kagami Rosylight 🏳️‍🌈🏳️‍⚧️ [:saschanaz] (they/them)]

Only depends on permissions api.

Comment 2

[Jan Jaeschke [:jjaschke]]

Attachment: Bug 2022043 - Fire PermissionStatus.onchange when Windows OS geolocation permission changes. r=saschanaz!

When the OS-level geolocation permission changes on Windows, PermissionStatus
objects for geolocation now fire their "change" event. This is done by
monitoring the wifiControl AppCapability's AccessChanged event in the parent
process and broadcasting the new state to all content processes via IPC.

The monitor starts lazily on the first geolocation permission query and
shuts down on xpcom-shutdown. The permission state mapping matches
Geolocation::GetLocationOSPermission().

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/6dc725ce279e
https://hg.mozilla.org/integration/autoland/rev/67ee617f2e0c
Fire PermissionStatus.onchange when Windows OS geolocation permission changes. r=saschanaz,win-reviewers,handyman,emilio

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/67ee617f2e0c