Commit bc3dddc2 authored by codrut.topliceanu's avatar codrut.topliceanu Committed by Arturo Mejia
Browse files

For #20367 - Fixes removeAllTabs adding to recently closed 

 `removeAllTabs` will now no longer allow `UndoMiddleware` from recovering the tabs.

(cherry picked from commit 2040af6a)
parent 0cd73a47

app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataController.kt

+1 −1
Original line number Diff line number Diff line
@@ -39,7 +39,7 @@ class DefaultDeleteBrowsingDataController( 


    override suspend fun deleteTabs() {

        withContext(coroutineContext) {

            removeAllTabs.invoke()

            removeAllTabs.invoke(false)

        }

    }

app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DefaultDeleteBrowsingDataControllerTest.kt

+1 −1
Original line number Diff line number Diff line
@@ -69,7 +69,7 @@ class DefaultDeleteBrowsingDataControllerTest { 
        controller.deleteTabs()



        verify {

            removeAllTabs.invoke()

            removeAllTabs.invoke(false)

        }

    }

app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuitTest.kt

+1 −1
Original line number Diff line number Diff line
@@ -75,7 +75,7 @@ class DeleteAndQuitTest { 


        verifyOrder {

            snackbar.show()

            removeAllTabsUseCases.invoke()

            removeAllTabsUseCases.invoke(false)

            activity.finishAndRemoveTask()

        }