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

Commit 80d0cf71 authored May 10, 2023 by Riddle Hsu Committed by Automerger Merge Worker May 10, 2023

Merge "Temporarily disable consecutive visibility assertion of close pip" into...

Merge "Temporarily disable consecutive visibility assertion of close pip" into udc-dev am: cafd5cd8

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

Change-Id: I674dda50adeb0ca8057ea314ef65935671ba690d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

parents 1dc1a655 cafd5cd8

libs/WindowManager/Shell/src/com/android/wm/shell/pip/PipTransition.java

+0 −3

Original line number	Diff line number	Diff line
		@@ -888,9 +888,6 @@ public class PipTransition extends PipTransitionController {
		// setting bounds.
		animator.setPipTransactionHandler(mTransactionConsumer).applySurfaceControlTransaction(
		leash, finishTransaction, PipAnimationController.FRACTION_END);
		// Remove the workaround after fixing ClosePipBySwipingDownTest that detects the shadow
		// as unexpected visible.
		finishTransaction.setShadowRadius(leash, 0);
		// Start to animate enter PiP.
		animator.setPipTransactionHandler(mPipOrganizer.getPipTransactionHandler()).start();
		}

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/ClosePipBySwipingDownTest.kt

+10 −0

Original line number	Diff line number	Diff line
		@@ -95,4 +95,14 @@ open class ClosePipBySwipingDownTest(flicker: FlickerTest) : ClosePipTransition(
		fun focusDoesNotChange() {
		flicker.assertEventLog { this.focusDoesNotChange() }
		}

		@Test
		override fun visibleLayersShownMoreThanOneConsecutiveEntry() {
		// TODO(b/270678766): Enable the assertion after fixing the case:
		// Assume the PiP task has shadow.
		// 1. The PiP activity is visible -> Task is invisible because it is occluded by activity.
		// 2. Activity becomes invisible -> Task is visible because it has shadow.
		// 3. Task is moved outside screen -> Task becomes invisible.
		// The assertion is triggered for 2 that the Task is only visible in one frame.
		}
		}