For mozilla-mobile#6832 - Only emit session changes if fragment is at…

…tached
ekager · May 21, 2020 · e60d17b · e60d17b
1 parent 7feae98
commit e60d17b
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@@ -72,6 +72,7 @@ import mozilla.components.support.ktx.kotlinx.coroutines.flow.ifChanged
 import org.mozilla.fenix.BrowserDirection
 import org.mozilla.fenix.HomeActivity
 import org.mozilla.fenix.R
+import org.mozilla.fenix.addons.runIfFragmentIsAttached
 import org.mozilla.fenix.browser.BrowserAnimator.Companion.getToolbarNavOptions
 import org.mozilla.fenix.browser.browsingmode.BrowsingMode
 import org.mozilla.fenix.components.FenixSnackbar
@@ -829,7 +830,9 @@ class HomeFragment : Fragment() {
     }
 
     private fun emitSessionChanges() {
-        homeFragmentStore.dispatch(HomeFragmentAction.TabsChange(getListOfTabs()))
+        runIfFragmentIsAttached {
+            homeFragmentStore.dispatch(HomeFragmentAction.TabsChange(getListOfTabs()))
+        }
     }
 
     private fun getListOfSessions(): List<Session> {