From 1e6bf12fdb2cefcbe173d74a3db54aa06421bc5b Mon Sep 17 00:00:00 2001
From: Pat Manning <patmanning@google.com>
Date: Wed, 13 Mar 2024 11:59:19 +0000
Subject: [PATCH] Set next page immediately on subsequent arrow/tab presses
 when navigating overivew.

This allows the user to scroll as quickly as they want.

Fix: 328749622
Test: manual
Flag: NONE
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:a9a11b47c3bff4d3ec4a4f300e12895d3235fe95)
Merged-In: I493841f11407e6fb9f15fd90b5a5e55fa4ed3ad3
Change-Id: I493841f11407e6fb9f15fd90b5a5e55fa4ed3ad3
---
 quickstep/src/com/android/quickstep/views/RecentsView.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/quickstep/src/com/android/quickstep/views/RecentsView.java b/quickstep/src/com/android/quickstep/views/RecentsView.java
index c333545b1f..1b2be4a6f6 100644
--- a/quickstep/src/com/android/quickstep/views/RecentsView.java
+++ b/quickstep/src/com/android/quickstep/views/RecentsView.java
@@ -4136,10 +4136,10 @@ public abstract class RecentsView<ACTIVITY_TYPE extends StatefulActivity<STATE_T
 
     private boolean snapToPageRelative(int delta, boolean cycle,
             @TaskGridNavHelper.TASK_NAV_DIRECTION int direction) {
-        // Ignore grid page snap events while scroll animations are running, otherwise the next
-        // page gets set before the animation finishes and can cause jumps.
+        // Set next page if scroll animation is still running, otherwise cannot snap to the
+        // next page on successive key presses. Setting the current page aborts the scroll.
         if (!mScroller.isFinished()) {
-            return true;
+            setCurrentPage(getNextPage());
         }
         int pageCount = getPageCount();
         if (pageCount == 0) {