From ccc28678d7d0365c594e2f5cde7ad81df6766892 Mon Sep 17 00:00:00 2001
From: Roman Tsisyk <roman@tsisyk.com>
Date: Mon, 19 Sep 2022 09:29:55 +0300
Subject: [PATCH] [android] Install window insets handlers in onSafeCreate()

This patch moves updateViewsInsets() from `onResume()` back to
`onSafeCreate()`. The #3389 issue happened because of premature
return from onSafeCreate() when app was lanunched with an intent:

```
    boolean isConsumed = savedInstanceState == null && processIntent(getIntent());
    boolean isFirstLaunch = Counters.isFirstLaunch(this);
    // If the map activity is launched by any incoming intent (deeplink, update maps event, etc)
    // or it's the first launch (onboarding) we haven't to try restoring the route,
    // showing the tips, etc.
    if (isConsumed || isFirstLaunch)
      return; <!-- Exited here.

    if (savedInstanceState == null && RoutingController.get().hasSavedRoute())
      addTask(new Factory.RestoreRouteTask());

    updateViewsInsets(); <!-- never reached if launched with an intent.
```

Follow up #3389

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>
---
 android/src/com/mapswithme/maps/MwmActivity.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/android/src/com/mapswithme/maps/MwmActivity.java b/android/src/com/mapswithme/maps/MwmActivity.java
index 29fb1e243f..0e512ac6e1 100644
--- a/android/src/com/mapswithme/maps/MwmActivity.java
+++ b/android/src/com/mapswithme/maps/MwmActivity.java
@@ -410,6 +410,7 @@ public class MwmActivity extends BaseMwmFragmentActivity
 
     boolean isLaunchByDeepLink = getIntent().getBooleanExtra(EXTRA_LAUNCH_BY_DEEP_LINK, false);
     initViews(isLaunchByDeepLink);
+    updateViewsInsets();
 
     boolean isConsumed = savedInstanceState == null && processIntent(getIntent());
     boolean isFirstLaunch = Counters.isFirstLaunch(this);
@@ -1042,8 +1043,6 @@ public class MwmActivity extends BaseMwmFragmentActivity
     mNavigationController.onActivityResumed(this);
     mMapButtonsController.onResume();
     mPlacePageController.onActivityResumed(this);
-
-    updateViewsInsets();
   }
 
   @Override