diff options
author | Kevin <kevhan@google.com> | 2019-04-30 12:51:32 -0700 |
---|---|---|
committer | Kevin <kevhan@google.com> | 2019-04-30 13:39:10 -0700 |
commit | 72e8025f7545cc8bc730a1df30e9216e2ce09de0 (patch) | |
tree | 50e1b6ef27f8d1bae5f58dc9196b6ff7d6d054d0 /go/quickstep | |
parent | d9c0fe130b2273bc012746fe21b34d47b06d4af5 (diff) | |
download | android_packages_apps_Trebuchet-72e8025f7545cc8bc730a1df30e9216e2ce09de0.tar.gz android_packages_apps_Trebuchet-72e8025f7545cc8bc730a1df30e9216e2ce09de0.tar.bz2 android_packages_apps_Trebuchet-72e8025f7545cc8bc730a1df30e9216e2ce09de0.zip |
Fix recents order being reversed sometimes
Occasionally, the recents list items were reversed. This is because
RecentsTaskList#getTasks returns the actual backing list which we were
reversing, so we solve this by making a new array list.
Bug: 131367388
Test: Do repro in bug, no longer gives reversed order
Change-Id: I825d284e4768c2b53fedf07480e99d195acfa6d6
Diffstat (limited to 'go/quickstep')
-rw-r--r-- | go/quickstep/src/com/android/quickstep/TaskListLoader.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/go/quickstep/src/com/android/quickstep/TaskListLoader.java b/go/quickstep/src/com/android/quickstep/TaskListLoader.java index 850c7e68a..1335cac53 100644 --- a/go/quickstep/src/com/android/quickstep/TaskListLoader.java +++ b/go/quickstep/src/com/android/quickstep/TaskListLoader.java @@ -80,7 +80,8 @@ public final class TaskListLoader { return; } // TODO: Look into error checking / more robust handling for when things go wrong. - mTaskListChangeId = mRecentsModel.getTasks(tasks -> { + mTaskListChangeId = mRecentsModel.getTasks(loadedTasks -> { + ArrayList<Task> tasks = new ArrayList<>(loadedTasks); // Reverse tasks to put most recent at the bottom of the view Collections.reverse(tasks); // Load task content |