From 375fc8f516083e878409ef68d4df242a17280827 Mon Sep 17 00:00:00 2001 From: Emily Kager Date: Tue, 28 May 2019 15:03:00 -0700 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/2110 - Pressing back on external link should close the tab --- .../java/org/mozilla/fenix/browser/BrowserFragment.kt | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt index 9aa24e8126..948e790761 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -555,10 +555,18 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope { readerViewFeature.onBackPressed() -> true customTabsIntegration.onBackPressed() -> true sessionFeature.onBackPressed() -> true - else -> false + else -> { + removeSessionIfNeeded() + false + } } } + private fun removeSessionIfNeeded() { + val session = getSessionById() ?: return + if (session.source == Session.Source.ACTION_VIEW) requireComponents.core.sessionManager.remove(session) + } + override fun onRequestPermissionsResult( requestCode: Int, permissions: Array,