Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 782e299e authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Fix |excludedDeskId| filter in removeActiveTask()" into main

parents 8faf21e6 17fc2e92

libs/WindowManager/Shell/src/com/android/wm/shell/desktopmode/DesktopRepository.kt

+1 −1
Original line number Diff line number Diff line
@@ -339,7 +339,7 @@ class DesktopRepository( 
        val affectedDisplays = mutableSetOf<Int>()

        desktopData

            .desksSequence()

            .filter { desk -> desk.displayId != excludedDeskId }

            .filter { desk -> desk.deskId != excludedDeskId }

            .forEach { desk ->

                val removed = removeActiveTaskFromDesk(desk.deskId, taskId, notifyListeners = false)

                if (removed) {

libs/WindowManager/Shell/tests/unittest/src/com/android/wm/shell/desktopmode/DesktopRepositoryTest.kt

+12 −0
Original line number Diff line number Diff line
@@ -274,6 +274,18 @@ class DesktopRepositoryTest(flags: FlagsParameterization) : ShellTestCase() { 
        assertThat(repo.isActiveTask(1)).isFalse()

    }



    @Test

    @EnableFlags(FLAG_ENABLE_MULTIPLE_DESKTOPS_BACKEND)

    fun removeActiveTask_excludingDesk_leavesTaskInDesk() {

        repo.addDesk(displayId = 2, deskId = 11)

        repo.addDesk(displayId = 3, deskId = 12)

        repo.addTaskToDesk(displayId = 3, deskId = 12, taskId = 100, isVisible = true)



        repo.removeActiveTask(taskId = 100, excludedDeskId = 12)



        assertThat(repo.getActiveTaskIdsInDesk(12)).contains(100)

    }



    @Test

    fun isActiveTask_nonExistingTask_returnsFalse() {

        assertThat(repo.isActiveTask(99)).isFalse()