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

Commit ade289c6 authored by Tiger Huang's avatar Tiger Huang Committed by Automerger Merge Worker
Browse files

Merge "Dispatch insets to client if mState is changed" into rvc-qpr-dev am:... 

Merge "Dispatch insets to client if mState is changed" into rvc-qpr-dev am: 046b604f am: c8f8eefe

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

Change-Id: I3e5d697405b17e4b8183e979324bb3793a735aeb
parents 08fb95d9 c8f8eefe

core/java/android/view/InsetsController.java

+10 −6
Original line number Diff line number Diff line
@@ -618,16 +618,20 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation 
            return false;

        }

        if (DEBUG) Log.d(TAG, "onStateChanged: " + state);

        updateState(state);



        boolean localStateChanged = !mState.equals(mLastDispatchedState,

                true /* excludingCaptionInsets */, true /* excludeInvisibleIme */);

        mLastDispatchedState.set(state, true /* copySources */);



        final InsetsState lastState = new InsetsState(mState, true /* copySources */);

        updateState(state);

        applyLocalVisibilityOverride();

        if (localStateChanged) {

            if (DEBUG) Log.d(TAG, "onStateChanged, notifyInsetsChanged, send state to WM: " + mState);



        if (!mState.equals(lastState, true /* excludingCaptionInsets */,

                true /* excludeInvisibleIme */)) {

            if (DEBUG) Log.d(TAG, "onStateChanged, notifyInsetsChanged");

            mHost.notifyInsetsChanged();

        }

        if (!mState.equals(state, true /* excludingCaptionInsets */,

                true /* excludeInvisibleIme */)) {

            if (DEBUG) Log.d(TAG, "onStateChanged, send state to WM: " + mState);

            updateRequestedState();

        }

        return true;

core/tests/coretests/src/android/view/InsetsControllerTest.java

+99 −2
Original line number Diff line number Diff line
@@ -27,6 +27,7 @@ import static android.view.InsetsState.ITYPE_NAVIGATION_BAR; 
import static android.view.InsetsState.ITYPE_STATUS_BAR;

import static android.view.ViewRootImpl.NEW_INSETS_MODE_FULL;

import static android.view.WindowInsets.Type.ime;

import static android.view.WindowInsets.Type.navigationBars;

import static android.view.WindowInsets.Type.statusBars;

import static android.view.WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE;
@@ -40,8 +41,11 @@ import static org.mockito.ArgumentMatchers.any; 
import static org.mockito.ArgumentMatchers.anyInt;

import static org.mockito.ArgumentMatchers.eq;

import static org.mockito.ArgumentMatchers.notNull;

import static org.mockito.Mockito.clearInvocations;

import static org.mockito.Mockito.mock;

import static org.mockito.Mockito.never;

import static org.mockito.Mockito.spy;

import static org.mockito.Mockito.times;

import static org.mockito.Mockito.verify;



import android.content.Context;
@@ -124,7 +128,7 @@ public class InsetsControllerTest { 
            }

            mTestClock = new OffsettableClock();

            mTestHandler = new TestHandler(null, mTestClock);

            mTestHost = new TestHost(mViewRoot);

            mTestHost = spy(new TestHost(mViewRoot));

            mController = new InsetsController(mTestHost, (controller, type) -> {

                if (type == ITYPE_IME) {

                    return new InsetsSourceConsumer(type, controller.getState(),
@@ -745,6 +749,99 @@ public class InsetsControllerTest { 
        });

    }



    @Test

    public void testInsetsChangedCount_controlSystemBars() {

        InstrumentationRegistry.getInstrumentation().runOnMainSync(() -> {

            prepareControls();



            // Hiding visible system bars should only causes insets change once for each bar.

            clearInvocations(mTestHost);

            mController.hide(statusBars() | navigationBars());

            verify(mTestHost, times(2)).notifyInsetsChanged();



            // Sending the same insets state should not cause insets change.

            // This simulates the callback from server after hiding system bars.

            clearInvocations(mTestHost);

            mController.onStateChanged(mController.getState());

            verify(mTestHost, never()).notifyInsetsChanged();



            // Showing invisible system bars should only causes insets change once for each bar.

            clearInvocations(mTestHost);

            mController.show(statusBars() | navigationBars());

            verify(mTestHost, times(2)).notifyInsetsChanged();



            // Sending the same insets state should not cause insets change.

            // This simulates the callback from server after showing system bars.

            clearInvocations(mTestHost);

            mController.onStateChanged(mController.getState());

            verify(mTestHost, never()).notifyInsetsChanged();

        });

    }



    @Test

    public void testInsetsChangedCount_controlIme() {

        InstrumentationRegistry.getInstrumentation().runOnMainSync(() -> {

            prepareControls();



            // Showing invisible ime should only causes insets change once.

            clearInvocations(mTestHost);

            mController.show(ime(), true /* fromIme */);

            verify(mTestHost, times(1)).notifyInsetsChanged();



            // Sending the same insets state should not cause insets change.

            // This simulates the callback from server after showing ime.

            clearInvocations(mTestHost);

            mController.onStateChanged(mController.getState());

            verify(mTestHost, never()).notifyInsetsChanged();



            // Hiding visible ime should only causes insets change once.

            clearInvocations(mTestHost);

            mController.hide(ime());

            verify(mTestHost, times(1)).notifyInsetsChanged();



            // Sending the same insets state should not cause insets change.

            // This simulates the callback from server after hiding ime.

            clearInvocations(mTestHost);

            mController.onStateChanged(mController.getState());

            verify(mTestHost, never()).notifyInsetsChanged();

        });

    }



    @Test

    public void testInsetsChangedCount_onStateChanged() {

        InstrumentationRegistry.getInstrumentation().runOnMainSync(() -> {

            final InsetsState localState = mController.getState();



            // Changing status bar frame should cause notifyInsetsChanged.

            clearInvocations(mTestHost);

            InsetsState newState = new InsetsState(localState, true /* copySources */);

            newState.getSource(ITYPE_STATUS_BAR).getFrame().bottom++;

            mController.onStateChanged(newState);

            verify(mTestHost, times(1)).notifyInsetsChanged();



            // Changing status bar visibility should cause notifyInsetsChanged.

            clearInvocations(mTestHost);

            newState = new InsetsState(localState, true /* copySources */);

            newState.getSource(ITYPE_STATUS_BAR).setVisible(false);

            mController.onStateChanged(newState);

            verify(mTestHost, times(1)).notifyInsetsChanged();



            // Changing invisible IME frame should not cause notifyInsetsChanged.

            clearInvocations(mTestHost);

            newState = new InsetsState(localState, true /* copySources */);

            newState.getSource(ITYPE_IME).getFrame().top--;

            mController.onStateChanged(newState);

            verify(mTestHost, never()).notifyInsetsChanged();



            // Changing IME visibility should cause notifyInsetsChanged.

            clearInvocations(mTestHost);

            newState = new InsetsState(localState, true /* copySources */);

            newState.getSource(ITYPE_IME).setVisible(true);

            mController.onStateChanged(newState);

            verify(mTestHost, times(1)).notifyInsetsChanged();

        });

    }



    private void waitUntilNextFrame() throws Exception {

        final CountDownLatch latch = new CountDownLatch(1);

        Choreographer.getMainThreadInstance().postCallback(Choreographer.CALLBACK_COMMIT,
@@ -777,7 +874,7 @@ public class InsetsControllerTest { 
        return controls;

    }



    private static class TestHost extends ViewRootInsetsControllerHost {

    public static class TestHost extends ViewRootInsetsControllerHost {



        private InsetsState mModifiedState = new InsetsState();