mirror of
https://gitlab.winehq.org/wine/wine-gecko.git
synced 2024-09-13 09:24:08 -07:00
868f93d340
GPS handling for Gonk abuses the Settings API in two ways. First, on mozsettings-changed event, it will trigger two new requests to read values (debug enabled and GPS locations ignore). This is useless since the event already contains the key that has changed and the new value, so there is no need to do a createLock().get() call. Then, in startup code, the Init() method is supposed to check itself whether it is already running. This is done through the mStarted boolean. The same Init() method is responsible for adding the mozsettings-changed observer, which is removed by the Shutdown() method. Investigation on device by using the Geolocation API has proven that we were leaking some observers. This is because checking mStarted boolean is performed after we request settings values and we install the mozsettings-changed observer. So any time the Init() gets called, we install it but we just remove it once in Shutdown(). |
||
---|---|---|
.. | ||
android | ||
gonk | ||
mac | ||
qt | ||
tests | ||
windows | ||
moz.build | ||
NetworkGeolocationProvider.js | ||
NetworkGeolocationProvider.manifest | ||
nsDeviceSensors.cpp | ||
nsDeviceSensors.h | ||
nsIOSFileConstantsService.idl | ||
OSFileConstants.cpp | ||
OSFileConstants.h |