RedAuburn/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

Merge pull request #5417 from alexzatsepin/MAPSME-3850-crash-in-first-run-mode

Browse source 
[android] Added 'mInFirsRun' checking before posting location updates…
This commit is contained in:
Aleksandr Zatsepin 2017-02-14 15:46:39 +03:00 • committed by GitHub
commit 1c7b0ae8d7
1 changed files with 12 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
android/src/com/mapswithme/maps/location/LocationHelper.java
View file

@ -49,6 +49,16 @@ public enum LocationHelper
@Override
public void onLocationUpdated(Location location)
{
// If we are still in the first run mode, i.e. user is staying on the first run screens,
// not on the map, we mustn't post location update to the core. Only this preserving allows us
// to play nice zoom animation once a user will leave first screens and will see a map.
if (mInFirstRun)
{
mLogger.d(TAG, "Location update is obtained and must be ignored, " +
"because the app is in a first run mode");
return;
}
mPredictor.onLocationUpdated(location);
nativeLocationUpdated(location.getTime(),
@ -257,15 +267,6 @@ public enum LocationHelper
return;
}
// If we are still in the first run mode, i.e. user stays on the first run screens,
// not on the map, we mustn't post location update to the core. Only this preserving allows us
// to play nice zoom animation once a user will see map.
if (mInFirstRun)
{
mLogger.d(TAG, "Location update is obtained, but ignore, because it's a first run mode");
return;
}
for (LocationListener listener : mListeners)
listener.onLocationUpdated(mSavedLocation);
mListeners.finishIterate();
@ -323,7 +324,8 @@ public enum LocationHelper
/**
* Registers listener about location changes.
* @param listener listener to register.
*
* @param listener listener to register.
* @param forceUpdate instantly notify given listener about available location, if any.
*/
@UiThread