diff options
author | Kevin <kevhan@google.com> | 2019-05-08 15:21:21 -0700 |
---|---|---|
committer | Kevin <kevhan@google.com> | 2019-05-08 15:30:33 -0700 |
commit | 41d3a2a1d7d249f42aef5dbb443b10f6337fd87c (patch) | |
tree | 128567c1f36b2280af94e98b90092e362e2731bd /go/quickstep/src | |
parent | 33a2946b7ff02d9e1281d4fd8b95e46fd8df6fa7 (diff) | |
download | android_packages_apps_Trebuchet-41d3a2a1d7d249f42aef5dbb443b10f6337fd87c.tar.gz android_packages_apps_Trebuchet-41d3a2a1d7d249f42aef5dbb443b10f6337fd87c.tar.bz2 android_packages_apps_Trebuchet-41d3a2a1d7d249f42aef5dbb443b10f6337fd87c.zip |
Fix bug skipping certain animations from ending
endAnimations forces an end to all running animations. This also removes
it from the list which changes the list while we're iterating over it.
This fixes the issue.
Bug: 132285006
Test: Manual test w/ build forcing endAnimations
Change-Id: I0ee9917866dafe00ef3122dd9a43259d19fb9f79
Diffstat (limited to 'go/quickstep/src')
-rw-r--r-- | go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java b/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java index 928234553..87ae6955e 100644 --- a/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java +++ b/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java @@ -250,7 +250,7 @@ public final class ContentFillItemAnimator extends SimpleItemAnimator { } mPendingAnims.remove(i); } - for (int i = 0; i < mRunningAnims.size(); i++) { + for (int i = mRunningAnims.size() - 1; i >= 0; i--) { ObjectAnimator anim = mRunningAnims.get(i); anim.end(); } |