🐞 MapboxFusedLocationEngineImpl may always report NETWORK_PROVIDER locations if the device clock is significantly faster than the satellite clock

[ystsoi]

Utils.isBetterLocation uses Location.getTime to compare location time. If the device clock (used by the NETWORK_PROVIDER) is significantly faster than the satellite clock (used by the GPS_PROVIDER), MapboxFusedLocationEngineImpl may always take the NETWORK_PROVIDER locations.

It is better to use Location.getElapsedRealtimeNanos instead, at least in API level 17.

There is a FUSED_PROVIDER in Android 12, which alleviates this issue. So better to test using older devices with cellular data enabled.