tree 50184a917284ce54fbe7eba01e73768388c6cd66
parent 709a52074d55654a6643f7a1a3c1ad454b7f4d86
author Tony Wickham <twickham@google.com> 1588622169 -0500
committer Tony Wickham <twickham@google.com> 1588698860 +0000

Fix continueComputingRecentsScrollIfNecessary() running infinitely

When continuing a gesture during quick switch, we cancel the
previous animation but don't invalidate it (since the gesture
isn't fully finished), and the new gesture handler calls
mRecentsView.setOnPageTransitionEndCallback(), so the previous
handler will never get STATE_RECENTS_SCROLLING_FINISHED. Thus,
continueComputingRecentsScrollIfNecessary() will never end.

To fix this, also check !mCanceled, since that is set when
continuing the gesture. Also use postOnAnimation() instead of
post() since that has better guarantees that the next
computation will wait for the next animation frame, rather
than whenever the message queue is available, which might
happen too frequently.

Test: quick switch from A to B and back to A before settling
on B, ensure we stop calling computeScroll().

Bug: 153589287
Change-Id: I6b4389ef2cf5a818438c7ae36d8807b3e41c8501
(cherry picked from commit 49b57e955c4cb37e91db77cc4813dd59eaacdc70)