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

Commit 2c1f1ad6 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "clearLockTask for root tasks as well at removal" into sc-dev am: ec801cc3 am: 3b51b818 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15006956

Change-Id: I836ea7bc7568b507b120d5c236a7bbc51e56a720
parents 8461a4e4 3b51b818

services/core/java/com/android/server/wm/Task.java

+1 −4
Original line number Diff line number Diff line
@@ -754,10 +754,7 @@ class Task extends TaskFragment { 
    }



    void removeIfPossible(String reason) {

        final boolean isRootTask = isRootTask();

        if (!isRootTask) {

        mAtmService.getLockTaskController().clearLockedTask(this);

        }

        if (shouldDeferRemoval()) {

            if (DEBUG_ROOT_TASK) Slog.i(TAG,

                    "removeTask:" + reason + " deferring removing taskId=" + mTaskId);

services/tests/wmtests/src/com/android/server/wm/TaskTests.java

+7 −4
Original line number Diff line number Diff line
@@ -60,6 +60,7 @@ import static org.mockito.ArgumentMatchers.anyBoolean; 
import static org.mockito.ArgumentMatchers.anyString;

import static org.mockito.ArgumentMatchers.eq;

import static org.mockito.ArgumentMatchers.same;

import static org.mockito.Mockito.atLeast;

import static org.mockito.Mockito.clearInvocations;

import static org.mockito.Mockito.never;
@@ -121,8 +122,8 @@ public class TaskTests extends WindowTestsBase { 


    @Test

    public void testRemoveContainer() {

        final Task taskController1 = createTask(mDisplayContent);

        final Task task = createTaskInRootTask(taskController1, 0 /* userId */);

        final Task rootTask = createTask(mDisplayContent);

        final Task task = createTaskInRootTask(rootTask, 0 /* userId */);

        final ActivityRecord activity = createActivityRecord(mDisplayContent, task);



        task.removeIfPossible();
@@ -130,12 +131,14 @@ public class TaskTests extends WindowTestsBase { 
        assertNull(task.getParent());

        assertEquals(0, task.getChildCount());

        assertNull(activity.getParent());

        verify(mAtm.getLockTaskController(), atLeast(1)).clearLockedTask(task);

        verify(mAtm.getLockTaskController(), atLeast(1)).clearLockedTask(rootTask);

    }



    @Test

    public void testRemoveContainer_deferRemoval() {

        final Task taskController1 = createTask(mDisplayContent);

        final Task task = createTaskInRootTask(taskController1, 0 /* userId */);

        final Task rootTask = createTask(mDisplayContent);

        final Task task = createTaskInRootTask(rootTask, 0 /* userId */);

        final ActivityRecord activity = createActivityRecord(mDisplayContent, task);



        doReturn(true).when(task).shouldDeferRemoval();