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

Commit 811a42d6 authored by Riddle Hsu's avatar Riddle Hsu Committed by Automerger Merge Worker
Browse files

Merge "Fix death recipient leakage of WindowTokenClient" into tm-dev am: 6cc80430 

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



Change-Id: I10ec3960b0062f694c3123eef31088fe21b068ed
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 7f7d0c33 6cc80430

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

+3 −0
Original line number Diff line number Diff line
@@ -116,6 +116,9 @@ class WindowContextListenerController { 
            return;

        }

        listener.unregister();

        if (listener.mDeathRecipient != null) {

            listener.mDeathRecipient.unlinkToDeath();

        }

    }



    void dispatchPendingConfigurationIfNeeded(int displayId) {

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

+6 −1
Original line number Diff line number Diff line
@@ -24,6 +24,7 @@ import static android.view.WindowManager.LayoutParams.TYPE_APPLICATION_OVERLAY; 
import static android.view.WindowManager.LayoutParams.TYPE_INPUT_METHOD_DIALOG;

import static android.window.WindowProvider.KEY_IS_WINDOW_PROVIDER_SERVICE;



import static com.android.dx.mockito.inline.extended.ExtendedMockito.mock;

import static com.android.dx.mockito.inline.extended.ExtendedMockito.spyOn;



import static com.google.common.truth.Truth.assertThat;
@@ -86,7 +87,7 @@ public class WindowContextListenerControllerTests extends WindowTestsBase { 


        assertEquals(1, mController.mListeners.size());



        final IBinder clientToken = new Binder();

        final IBinder clientToken = mock(IBinder.class);

        mController.registerWindowContainerListener(clientToken, mContainer, -1,

                TYPE_APPLICATION_OVERLAY, null /* options */);
@@ -103,6 +104,10 @@ public class WindowContextListenerControllerTests extends WindowTestsBase { 
        WindowContextListenerController.WindowContextListenerImpl listener =

                mController.mListeners.get(mClientToken);

        assertEquals(container, listener.getWindowContainer());



        mController.unregisterWindowContainerListener(clientToken);

        assertFalse(mController.mListeners.containsKey(clientToken));

        verify(clientToken).unlinkToDeath(any(), anyInt());

    }



    @UseTestDisplay