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

Commit 4d576bfa authored by Riddle Hsu's avatar Riddle Hsu
Browse files

Use normal rotation if orientation is changed by resume-scheduled app 

With traditional normal rotation, if an app calls
Activity#setRequestedOrientation to change orientation in its onCreate
or onResume, the system display info will be updated immediately.
So if the app gets display rotation right after the orientation request,
it can get the rotated result.

While if fixed rotation transform takes effect for the case, though the
rotated display adjustments will be sent to the app, it is too late
that the app may have accessed the global display info which is not
rotated, that may cause inconsistent UI layout if the app depends on
the rotation info.

Bug: 167564038
Test: DisplayContentTests#testNoFixedRotationOnResumedScheduledApp
Change-Id: I3253264cb1bb8886074e88012582daa3589a1896
parent e9dfe713

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

+9 −0
Original line number Diff line number Diff line
@@ -1465,6 +1465,15 @@ class DisplayContent extends RootDisplayArea implements WindowManagerPolicy.Disp 
                // intermediate orientation change, it is more stable to freeze the display.

                return false;

            }

            if (r.isState(RESUMED) && !r.getRootTask().mInResumeTopActivity) {

                // If the activity is executing or has done the lifecycle callback, use normal

                // rotation animation so the display info can be updated immediately (see

                // updateDisplayAndOrientation). This prevents a compatibility issue such as

                // calling setRequestedOrientation in Activity#onCreate and then get display info.

                // If fixed rotation is applied, the display rotation will still be the old one,

                // unless the client side gets the rotation again after the adjustments arrive.

                return false;

            }

        } else if (r != topRunningActivity()) {

            // If the transition has not started yet, the activity must be the top.

            return false;

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

+21 −0
Original line number Diff line number Diff line
@@ -1348,6 +1348,27 @@ public class DisplayContentTests extends WindowTestsBase { 
        assertTrue(displayContent.getPinnedStackController().isPipActiveOrWindowingModeChanging());

    }



    @Test

    public void testNoFixedRotationOnResumedScheduledApp() {

        unblockDisplayRotation(mDisplayContent);

        final ActivityRecord app = new ActivityBuilder(mAtm).setCreateTask(true).build();

        app.setVisible(false);

        app.setState(Task.ActivityState.RESUMED, "test");

        mDisplayContent.prepareAppTransition(WindowManager.TRANSIT_ACTIVITY_OPEN,

                false /* alwaysKeepCurrent */);

        mDisplayContent.mOpeningApps.add(app);

        final int newOrientation = getRotatedOrientation(mDisplayContent);

        app.setRequestedOrientation(newOrientation);



        // The condition should reject using fixed rotation because the resumed client in real case

        // might get display info immediately. And the fixed rotation adjustments haven't arrived

        // client side so the info may be inconsistent with the requested orientation.

        verify(mDisplayContent).handleTopActivityLaunchingInDifferentOrientation(eq(app),

                eq(true) /* checkOpening */);

        assertFalse(app.isFixedRotationTransforming());

        assertFalse(mDisplayContent.hasTopFixedRotationLaunchingApp());

    }



    @Test

    public void testRecentsNotRotatingWithFixedRotation() {

        unblockDisplayRotation(mDisplayContent);