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

Commit 095b6e7c authored by Nicolò Mazzucato's avatar Nicolò Mazzucato Committed by Automerger Merge Worker
Browse files

Merge "Revert "Add a delay to rotation updates to help avoid fold state race""... 

Merge "Revert "Add a delay to rotation updates to help avoid fold state race"" into tm-qpr-dev am: c4b1be4d am: 3a00521a am: 166086e1 am: 7e5241f7

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



Change-Id: Ia6a3ace917fde20b304dff622f661c2260f216f2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 53c3655d 7e5241f7

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

+5 −7
Original line number Diff line number Diff line
@@ -87,8 +87,6 @@ import java.util.Set; 
 */

public class DisplayRotation {

    private static final String TAG = TAG_WITH_CLASS_NAME ? "DisplayRotation" : TAG_WM;

    // Delay to avoid race between fold update and orientation update.

    private static final int ORIENTATION_UPDATE_DELAY_MS = 800;



    // Delay in milliseconds when updating config due to folding events. This prevents

    // config changes and unexpected jumps while folding the device to closed state.
@@ -1789,15 +1787,15 @@ public class DisplayRotation { 
                mDeviceState = newState;

                // Now mFoldState is set to HALF_FOLDED, the overrideFrozenRotation function will

                // return true, so rotation is unlocked.

                mService.updateRotation(false /* alwaysSendConfiguration */,

                        false /* forceRelayout */);

            } else {

                mInHalfFoldTransition = true;

                mDeviceState = newState;

            }

            UiThread.getHandler().postDelayed(

                    () -> {

                // Tell the device to update its orientation.

                mService.updateRotation(false /* alwaysSendConfiguration */,

                        false /* forceRelayout */);

                    }, ORIENTATION_UPDATE_DELAY_MS);

            }

            // Alert the activity of possible new bounds.

            UiThread.getHandler().removeCallbacks(mActivityBoundsUpdateCallback);

            UiThread.getHandler().postDelayed(mActivityBoundsUpdateCallback,

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

+2 −2
Original line number Diff line number Diff line
@@ -792,7 +792,7 @@ public class DisplayRotationTests { 
        // ... until half-fold

        mTarget.foldStateChanged(DeviceStateController.DeviceState.HALF_FOLDED);

        assertTrue(waitForUiHandler());

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

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

        assertTrue(waitForUiHandler());

        assertEquals(Surface.ROTATION_0, mTarget.rotationForOrientation(

                SCREEN_ORIENTATION_UNSPECIFIED, Surface.ROTATION_0));
@@ -800,7 +800,7 @@ public class DisplayRotationTests { 
        // ... then transition back to flat

        mTarget.foldStateChanged(DeviceStateController.DeviceState.OPEN);

        assertTrue(waitForUiHandler());

        verify(sMockWm, atLeast(1)).updateRotation(anyBoolean(), anyBoolean());

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

        assertTrue(waitForUiHandler());

        assertEquals(Surface.ROTATION_270, mTarget.rotationForOrientation(

                SCREEN_ORIENTATION_UNSPECIFIED, Surface.ROTATION_0));