| commit | 41d3a2a1d7d249f42aef5dbb443b10f6337fd87c | [log] [tgz] |
|---|---|---|
| author | Kevin <kevhan@google.com> | Wed May 08 15:21:21 2019 -0700 |
| committer | Kevin <kevhan@google.com> | Wed May 08 15:30:33 2019 -0700 |
| tree | 128567c1f36b2280af94e98b90092e362e2731bd | |
| parent | 33a2946b7ff02d9e1281d4fd8b95e46fd8df6fa7 [diff] |
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
diff --git a/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java b/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java index 9282345..87ae695 100644 --- a/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java +++ b/go/quickstep/src/com/android/quickstep/ContentFillItemAnimator.java
@@ -250,7 +250,7 @@ } 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(); }