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

Commit 390a57f0 authored by Mariia Sandrikova's avatar Mariia Sandrikova Committed by Automerger Merge Worker
Browse files

Merge "Added OnSetRequestedOrientation callback" into udc-dev am: 5f9de503 am: 9b917e7c 

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



Change-Id: Ia2bdb7b538f4f7a7009c67af5cfb433fbd719fdc
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents d521c9c6 9b917e7c

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

+2 −0
Original line number Diff line number Diff line
@@ -7917,6 +7917,8 @@ final class ActivityRecord extends WindowToken implements WindowManagerService.A 


        mAtmService.getTaskChangeNotificationController().notifyActivityRequestedOrientationChanged(

                task.mTaskId, requestedOrientation);



        mDisplayContent.getDisplayRotation().onSetRequestedOrientation();

    }



    /*

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

+22 −0
Original line number Diff line number Diff line
@@ -97,6 +97,8 @@ public class DisplayRotation { 
    // config changes and unexpected jumps while folding the device to closed state.

    private static final int FOLDING_RECOMPUTE_CONFIG_DELAY_MS = 800;



    private static final int ROTATION_UNDEFINED = -1;



    private static class RotationAnimationPair {

        @AnimRes

        int mEnter;
@@ -189,6 +191,12 @@ public class DisplayRotation { 
     */

    private int mShowRotationSuggestions;



    /**

     * The most recent {@link Surface.Rotation} choice shown to the user for confirmation, or

     * {@link #ROTATION_UNDEFINED}

     */

    private int mRotationChoiceShownToUserForConfirmation = ROTATION_UNDEFINED;



    private static final int ALLOW_ALL_ROTATIONS_UNDEFINED = -1;

    private static final int ALLOW_ALL_ROTATIONS_DISABLED = 0;

    private static final int ALLOW_ALL_ROTATIONS_ENABLED = 1;
@@ -894,6 +902,7 @@ public class DisplayRotation { 


    @VisibleForTesting

    void setUserRotation(int userRotationMode, int userRotation) {

        mRotationChoiceShownToUserForConfirmation = ROTATION_UNDEFINED;

        if (isDefaultDisplay) {

            // We'll be notified via settings listener, so we don't need to update internal values.

            final ContentResolver res = mContext.getContentResolver();
@@ -1613,6 +1622,17 @@ public class DisplayRotation { 
        }

    }



    /**

     * Called from {@link ActivityRecord#setRequestedOrientation(int)}

     */

    void onSetRequestedOrientation() {

        if (mCompatPolicyForImmersiveApps == null

                || mRotationChoiceShownToUserForConfirmation == ROTATION_UNDEFINED) {

            return;

        }

        mOrientationListener.onProposedRotationChanged(mRotationChoiceShownToUserForConfirmation);

    }



    void dump(String prefix, PrintWriter pw) {

        pw.println(prefix + "DisplayRotation");

        pw.println(prefix + "  mCurrentAppOrientation="
@@ -2012,9 +2032,11 @@ public class DisplayRotation { 
            mService.mPowerManagerInternal.setPowerBoost(Boost.INTERACTION, 0);

            dispatchProposedRotation(rotation);

            if (isRotationChoiceAllowed(rotation)) {

                mRotationChoiceShownToUserForConfirmation = rotation;

                final boolean isValid = isValidRotationChoice(rotation);

                sendProposedRotationChangeToStatusBarInternal(rotation, isValid);

            } else {

                mRotationChoiceShownToUserForConfirmation = ROTATION_UNDEFINED;

                mService.updateRotation(false /* alwaysSendConfiguration */,

                        false /* forceRelayout */);

            }

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

+6 −0
Original line number Diff line number Diff line
@@ -589,12 +589,18 @@ public class ActivityRecordTests extends WindowTestsBase { 
                throw new IllegalStateException("Orientation in new config should be either"

                        + "landscape or portrait.");

        }



        final DisplayRotation displayRotation = activity.mDisplayContent.getDisplayRotation();

        spyOn(displayRotation);



        activity.setRequestedOrientation(requestedOrientation);



        final ActivityConfigurationChangeItem expected =

                ActivityConfigurationChangeItem.obtain(newConfig);

        verify(mAtm.getLifecycleManager()).scheduleTransaction(eq(activity.app.getThread()),

                eq(activity.token), eq(expected));



        verify(displayRotation).onSetRequestedOrientation();

    }



    @Test

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

+38 −3
Original line number Diff line number Diff line
@@ -70,6 +70,7 @@ import android.view.IRotationWatcher; 
import android.view.Surface;

import android.view.WindowManager;



import androidx.annotation.Nullable;

import androidx.test.filters.SmallTest;



import com.android.internal.util.test.FakeSettingsProvider;
@@ -140,6 +141,8 @@ public class DisplayRotationTests { 


    private DeviceStateController mDeviceStateController;

    private TestDisplayRotation mTarget;

    @Nullable

    private DisplayRotationImmersiveAppCompatPolicy mDisplayRotationImmersiveAppCompatPolicyMock;



    @BeforeClass

    public static void setUpOnce() {
@@ -165,7 +168,7 @@ public class DisplayRotationTests { 
        LocalServices.removeServiceForTest(StatusBarManagerInternal.class);

        mMockStatusBarManagerInternal = mock(StatusBarManagerInternal.class);

        LocalServices.addService(StatusBarManagerInternal.class, mMockStatusBarManagerInternal);



        mDisplayRotationImmersiveAppCompatPolicyMock = null;

        mBuilder = new DisplayRotationBuilder();

    }
@@ -577,6 +580,38 @@ public class DisplayRotationTests { 
        verify(mMockStatusBarManagerInternal).onProposedRotationChanged(Surface.ROTATION_90, true);

    }



    @Test

    public void testNotifiesChoiceWhenSensorUpdates_immersiveApp() throws Exception {

        mDisplayRotationImmersiveAppCompatPolicyMock = mock(

                DisplayRotationImmersiveAppCompatPolicy.class);

        when(mDisplayRotationImmersiveAppCompatPolicyMock.isRotationLockEnforced(

                Surface.ROTATION_90)).thenReturn(true);



        mBuilder.build();

        configureDisplayRotation(SCREEN_ORIENTATION_PORTRAIT, false, false);



        thawRotation();



        enableOrientationSensor();



        mOrientationSensorListener.onSensorChanged(createSensorEvent(Surface.ROTATION_90));

        assertTrue(waitForUiHandler());



        verify(mMockStatusBarManagerInternal).onProposedRotationChanged(Surface.ROTATION_90, true);



        // An imaginary ActivityRecord.setRequestedOrientation call disables immersive mode:

        when(mDisplayRotationImmersiveAppCompatPolicyMock.isRotationLockEnforced(

                Surface.ROTATION_90)).thenReturn(false);



        // And then ActivityRecord.setRequestedOrientation calls onSetRequestedOrientation.

        mTarget.onSetRequestedOrientation();



        // onSetRequestedOrientation should lead to a second call to

        // mOrientationListener.onProposedRotationChanged

        // but now, instead of notifying mMockStatusBarManagerInternal, it calls updateRotation:

        verify(sMockWm).updateRotation(false, false);

    }



    @Test

    public void testAllowAllRotations_allowsUpsideDownSuggestion()

            throws Exception {
@@ -1404,7 +1439,7 @@ public class DisplayRotationTests { 
        }

    }



    private static class TestDisplayRotation extends DisplayRotation {

    private class TestDisplayRotation extends DisplayRotation {

        IntConsumer mProposedRotationCallback;



        TestDisplayRotation(DisplayContent dc, DisplayAddress address, DisplayPolicy policy,
@@ -1417,7 +1452,7 @@ public class DisplayRotationTests { 
        @Override

        DisplayRotationImmersiveAppCompatPolicy initImmersiveAppCompatPolicy(

                WindowManagerService service, DisplayContent displayContent) {

            return null;

            return mDisplayRotationImmersiveAppCompatPolicyMock;

        }



        @Override