Loading services/core/java/com/android/server/wm/LetterboxUiController.java +2 −7 Original line number Diff line number Diff line Loading @@ -1373,7 +1373,8 @@ final class LetterboxUiController { } final boolean shouldShowLetterboxUi = (mActivityRecord.isInLetterboxAnimation() || isSurfaceVisible(mainWindow)) (mActivityRecord.isInLetterboxAnimation() || mActivityRecord.isVisible() || mActivityRecord.isVisibleRequested()) && mainWindow.areAppWindowBoundsLetterboxed() // Check for FLAG_SHOW_WALLPAPER explicitly instead of using // WindowContainer#showWallpaper because the later will return true when this Loading @@ -1385,12 +1386,6 @@ final class LetterboxUiController { return shouldShowLetterboxUi; } @VisibleForTesting boolean isSurfaceVisible(WindowState mainWindow) { return mainWindow.isOnScreen() && (mActivityRecord.isVisible() || mActivityRecord.isVisibleRequested()); } private Color getLetterboxBackgroundColor() { final WindowState w = mActivityRecord.findMainWindow(); if (w == null || w.isLetterboxedForDisplayCutout()) { Loading services/tests/wmtests/src/com/android/server/wm/LetterboxUiControllerTest.java +2 −1 Original line number Diff line number Diff line Loading @@ -609,7 +609,8 @@ public class LetterboxUiControllerTest extends WindowTestsBase { doReturn(false).when(mActivity).isInLetterboxAnimation(); assertEquals(expectedRadius, mController.getRoundedCornersRadius(mainWindow)); doReturn(false).when(mainWindow).isOnScreen(); doReturn(false).when(mActivity).isVisibleRequested(); doReturn(false).when(mActivity).isVisible(); assertEquals(0, mController.getRoundedCornersRadius(mainWindow)); doReturn(true).when(mActivity).isInLetterboxAnimation(); Loading services/tests/wmtests/src/com/android/server/wm/SizeCompatTests.java +1 −2 Original line number Diff line number Diff line Loading @@ -909,8 +909,7 @@ public class SizeCompatTests extends WindowTestsBase { assertEquals(window, mActivity.findMainWindow()); spyOn(mActivity.mLetterboxUiController); doReturn(true).when(mActivity.mLetterboxUiController) .isSurfaceVisible(any()); doReturn(true).when(mActivity).isVisibleRequested(); assertTrue(mActivity.mLetterboxUiController.shouldShowLetterboxUi( mActivity.findMainWindow())); Loading Loading
services/core/java/com/android/server/wm/LetterboxUiController.java +2 −7 Original line number Diff line number Diff line Loading @@ -1373,7 +1373,8 @@ final class LetterboxUiController { } final boolean shouldShowLetterboxUi = (mActivityRecord.isInLetterboxAnimation() || isSurfaceVisible(mainWindow)) (mActivityRecord.isInLetterboxAnimation() || mActivityRecord.isVisible() || mActivityRecord.isVisibleRequested()) && mainWindow.areAppWindowBoundsLetterboxed() // Check for FLAG_SHOW_WALLPAPER explicitly instead of using // WindowContainer#showWallpaper because the later will return true when this Loading @@ -1385,12 +1386,6 @@ final class LetterboxUiController { return shouldShowLetterboxUi; } @VisibleForTesting boolean isSurfaceVisible(WindowState mainWindow) { return mainWindow.isOnScreen() && (mActivityRecord.isVisible() || mActivityRecord.isVisibleRequested()); } private Color getLetterboxBackgroundColor() { final WindowState w = mActivityRecord.findMainWindow(); if (w == null || w.isLetterboxedForDisplayCutout()) { Loading
services/tests/wmtests/src/com/android/server/wm/LetterboxUiControllerTest.java +2 −1 Original line number Diff line number Diff line Loading @@ -609,7 +609,8 @@ public class LetterboxUiControllerTest extends WindowTestsBase { doReturn(false).when(mActivity).isInLetterboxAnimation(); assertEquals(expectedRadius, mController.getRoundedCornersRadius(mainWindow)); doReturn(false).when(mainWindow).isOnScreen(); doReturn(false).when(mActivity).isVisibleRequested(); doReturn(false).when(mActivity).isVisible(); assertEquals(0, mController.getRoundedCornersRadius(mainWindow)); doReturn(true).when(mActivity).isInLetterboxAnimation(); Loading
services/tests/wmtests/src/com/android/server/wm/SizeCompatTests.java +1 −2 Original line number Diff line number Diff line Loading @@ -909,8 +909,7 @@ public class SizeCompatTests extends WindowTestsBase { assertEquals(window, mActivity.findMainWindow()); spyOn(mActivity.mLetterboxUiController); doReturn(true).when(mActivity.mLetterboxUiController) .isSurfaceVisible(any()); doReturn(true).when(mActivity).isVisibleRequested(); assertTrue(mActivity.mLetterboxUiController.shouldShowLetterboxUi( mActivity.findMainWindow())); Loading