From 12b12f7a591dc8f8a22d36da1a712bbbfdcef0b5 Mon Sep 17 00:00:00 2001 From: Garfield Tan Date: Fri, 22 Feb 2019 16:33:27 -0800 Subject: Update comments related to force default rotation. Bug: 124420570 Test: TH is enough. Change-Id: I00058bb932ef16db2b49b494156c2ca1c650ce1c --- services/core/java/com/android/server/wm/DisplayRotation.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/services/core/java/com/android/server/wm/DisplayRotation.java b/services/core/java/com/android/server/wm/DisplayRotation.java index c6b7060fe649..34a480291863 100644 --- a/services/core/java/com/android/server/wm/DisplayRotation.java +++ b/services/core/java/com/android/server/wm/DisplayRotation.java @@ -236,16 +236,13 @@ public class DisplayRotation { } mDemoRotationLock = SystemProperties.getBoolean("persist.demo.rotationlock", false); - // Only force the default orientation if the screen is xlarge, at least 960dp x 720dp, per - // http://developer.android.com/guide/practices/screens_support.html#range - // For car, ignore the dp limitation. It's physically impossible to rotate the car's screen - // so if the orientation is forced, we need to respect that no matter what. + // It's physically impossible to rotate the car's screen. final boolean isCar = mContext.getPackageManager().hasSystemFeature( PackageManager.FEATURE_AUTOMOTIVE); - // For TV, it's usually 960dp x 540dp, ignore the size limitation. - // so if the orientation is forced, we need to respect that no matter what. + // It's also not likely to rotate a TV screen. final boolean isTv = mContext.getPackageManager().hasSystemFeature( PackageManager.FEATURE_LEANBACK); + // Not much of use to rotate the display since it's close to square. final boolean isCloseToSquare = isNonDecorDisplayCloseToSquare(Surface.ROTATION_0, width, height); final boolean forceDesktopMode = -- cgit v1.2.3-59-g8ed1b