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

Commit 14c9e89e authored by Mady Mellor's avatar Mady Mellor Committed by Automerger Merge Worker
Browse files

Merge "Protect against NPE when updating state" into udc-dev am: ec5540c7... 

Merge "Protect against NPE when updating state" into udc-dev am: ec5540c7 am: 182ed04f am: 00cca4af

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



Change-Id: I3789de89837e8338be4047d9d26c62cd036c7841
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 7f0e507c 00cca4af

libs/WindowManager/Shell/src/com/android/wm/shell/taskview/TaskViewTransitions.java

+4 −1
Original line number Diff line number Diff line
@@ -202,6 +202,7 @@ public class TaskViewTransitions implements Transitions.TransitionHandler { 
    }



    void setTaskViewVisible(TaskViewTaskController taskView, boolean visible) {

        if (mTaskViews.get(taskView) == null) return;

        if (mTaskViews.get(taskView).mVisible == visible) return;

        if (taskView.getTaskInfo() == null) {

            // Nothing to update, task is not yet available
@@ -220,17 +221,19 @@ public class TaskViewTransitions implements Transitions.TransitionHandler { 


    void updateBoundsState(TaskViewTaskController taskView, Rect boundsOnScreen) {

        TaskViewRequestedState state = mTaskViews.get(taskView);

        if (state == null) return;

        state.mBounds.set(boundsOnScreen);

    }



    void updateVisibilityState(TaskViewTaskController taskView, boolean visible) {

        TaskViewRequestedState state = mTaskViews.get(taskView);

        if (state == null) return;

        state.mVisible = visible;

    }



    void setTaskBounds(TaskViewTaskController taskView, Rect boundsOnScreen) {

        TaskViewRequestedState state = mTaskViews.get(taskView);

        if (Objects.equals(boundsOnScreen, state.mBounds)) {

        if (state == null || Objects.equals(boundsOnScreen, state.mBounds)) {

            return;

        }

        state.mBounds.set(boundsOnScreen);

libs/WindowManager/Shell/tests/unittest/src/com/android/wm/shell/taskview/TaskViewTransitionsTest.java

+19 −0
Original line number Diff line number Diff line
@@ -179,4 +179,23 @@ public class TaskViewTransitionsTest extends ShellTestCase { 
                mTaskViewTransitions.findPending(mTaskViewTaskController, TRANSIT_CHANGE);

        assertThat(pendingBounds2).isNull();

    }



    @Test

    public void testSetTaskVisibility_taskRemoved_noNPE() {

        mTaskViewTransitions.removeTaskView(mTaskViewTaskController);



        assumeTrue(Transitions.ENABLE_SHELL_TRANSITIONS);



        mTaskViewTransitions.setTaskViewVisible(mTaskViewTaskController, false);

    }



    @Test

    public void testSetTaskBounds_taskRemoved_noNPE() {

        mTaskViewTransitions.removeTaskView(mTaskViewTaskController);



        assumeTrue(Transitions.ENABLE_SHELL_TRANSITIONS);



        mTaskViewTransitions.setTaskBounds(mTaskViewTaskController,

                new Rect(0, 0, 100, 100));

    }

}