Merge "Allow PipTouchHandler to re-enable the touches" into tm-qpr-dev am: 247ad78eaf am: d3f39b4ea1
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22211152 Change-Id: Iaf21712789d35f1e4262fa6f98868787bb84ff97 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -210,7 +210,7 @@ public class PipAnimationController {
|
||||
/**
|
||||
* Quietly cancel the animator by removing the listeners first.
|
||||
*/
|
||||
public static void quietCancel(@NonNull ValueAnimator animator) {
|
||||
static void quietCancel(@NonNull ValueAnimator animator) {
|
||||
animator.removeAllUpdateListeners();
|
||||
animator.removeAllListeners();
|
||||
animator.cancel();
|
||||
|
||||
@@ -145,10 +145,12 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener,
|
||||
// These callbacks are called on the update thread
|
||||
private final PipAnimationController.PipAnimationCallback mPipAnimationCallback =
|
||||
new PipAnimationController.PipAnimationCallback() {
|
||||
private boolean mIsCancelled;
|
||||
@Override
|
||||
public void onPipAnimationStart(TaskInfo taskInfo,
|
||||
PipAnimationController.PipTransitionAnimator animator) {
|
||||
final int direction = animator.getTransitionDirection();
|
||||
mIsCancelled = false;
|
||||
sendOnPipTransitionStarted(direction);
|
||||
}
|
||||
|
||||
@@ -156,6 +158,10 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener,
|
||||
public void onPipAnimationEnd(TaskInfo taskInfo, SurfaceControl.Transaction tx,
|
||||
PipAnimationController.PipTransitionAnimator animator) {
|
||||
final int direction = animator.getTransitionDirection();
|
||||
if (mIsCancelled) {
|
||||
sendOnPipTransitionFinished(direction);
|
||||
return;
|
||||
}
|
||||
final int animationType = animator.getAnimationType();
|
||||
final Rect destinationBounds = animator.getDestinationBounds();
|
||||
if (isInPipDirection(direction) && animator.getContentOverlayLeash() != null) {
|
||||
@@ -194,6 +200,7 @@ public class PipTaskOrganizer implements ShellTaskOrganizer.TaskListener,
|
||||
public void onPipAnimationCancel(TaskInfo taskInfo,
|
||||
PipAnimationController.PipTransitionAnimator animator) {
|
||||
final int direction = animator.getTransitionDirection();
|
||||
mIsCancelled = true;
|
||||
if (isInPipDirection(direction) && animator.getContentOverlayLeash() != null) {
|
||||
fadeOutAndRemoveOverlay(animator.getContentOverlayLeash(),
|
||||
animator::clearContentOverlay, true /* withStartDelay */);
|
||||
|
||||
@@ -783,7 +783,7 @@ public class PipController implements PipTransitionController.PipTransitionCallb
|
||||
mPipAnimationController.getCurrentAnimator();
|
||||
if (animator != null && animator.isRunning()) {
|
||||
// cancel any running animator, as it is using stale display layout information
|
||||
PipAnimationController.quietCancel(animator);
|
||||
animator.cancel();
|
||||
}
|
||||
onDisplayChangedUncheck(layout, saveRestoreSnapFraction);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user