No Issue - Capture session manager in scope before creating the delete operation

5 years ago · 90bdbd04e0
parent ac5c4a0f79
commit 90bdbd04e0
1 changed files with 3 additions and 0 deletions
--- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@ -567,6 +567,7 @@ class HomeFragment : Fragment(), CoroutineScope, AccountObserver {
    }

    private fun removeAllTabsWithUndo(isPrivate: Boolean) {
+        val sessionManager = requireComponents.core.sessionManager
        val useCases = context?.components?.useCases?.tabsUseCases ?: return

        getManagedEmitter<SessionControlChange>().onNext(SessionControlChange.TabsChange(listOf()))
@ -591,6 +592,8 @@ class HomeFragment : Fragment(), CoroutineScope, AccountObserver {
    }

    private fun removeTabWithUndo(sessionId: String) {
+        val sessionManager = requireComponents.core.sessionManager
+
        val deleteOperation: (suspend () -> Unit) = {
            sessionManager.findSessionById(sessionId)
                ?.let { session ->