Closes mozilla-mobile#614: Fix Custom Tab shows blank screen on restore

jonalmeida · Aug 29, 2019 · 3f94619 · 3f94619
1 parent f5a6619
commit 3f94619
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/app/src/main/java/org/mozilla/reference/browser/browser/CustomTabsIntegration.kt b/app/src/main/java/org/mozilla/reference/browser/browser/CustomTabsIntegration.kt
@@ -18,6 +18,7 @@ import mozilla.components.feature.customtabs.CustomTabsToolbarFeature
 import mozilla.components.feature.session.SessionUseCases
 import mozilla.components.support.base.feature.BackHandler
 import mozilla.components.support.base.feature.LifecycleAwareFeature
+import mozilla.components.support.base.log.logger.Logger
 import org.mozilla.reference.browser.BrowserActivity
 import org.mozilla.reference.browser.R
 import org.mozilla.reference.browser.ext.share
@@ -32,12 +33,17 @@ class CustomTabsIntegration(
     activity: Activity?
 ) : LifecycleAwareFeature, BackHandler {
 
+    private val session = sessionManager.findSessionById(sessionId)
+    private val logger = Logger("CustomTabsIntegration")
+
     init {
+        if (session == null) {
+            logger.warn("The session for this ID, no longer exists. Finishing activity.")
+            activity?.finish()
+        }
         toolbar.urlBoxView = null
     }
 
-    private val session = sessionManager.findSessionById(sessionId)
-
     private val menuToolbar by lazy {
         val forward = BrowserMenuItemToolbar.Button(
             mozilla.components.ui.icons.R.drawable.mozac_ic_forward,