Import PR 3500 from ekager

No Issue - Capture session manager in scope before creating the delete operation
1 changed files with 4 additions and 2 deletions
--- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@ -567,16 +567,16 @@ 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()))

        val deleteOperation: (suspend () -> Unit) = {
-            sessionManager.filteredSessions(isPrivate) { it.id == pendingSessionDeletion?.sessionId }.forEach {
+            sessionManager.sessions.filter { it.private == isPrivate }.forEach {
                useCases.removeTab.invoke(it)
            }
        }
-
        deleteAllSessionsJob = deleteOperation

        allowUndo(
@ -591,6 +591,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 ->
Author	SHA1	Message	Date
Mihai Tabara	494a78db4d	Import PR 3500 from ekager	5 years ago
Jeff Boek	31e26a20f7	No Issue - Capture session manager in scope before creating the delete operation	5 years ago