From 72e8025f7545cc8bc730a1df30e9216e2ce09de0 Mon Sep 17 00:00:00 2001 From: Kevin Date: Tue, 30 Apr 2019 12:51:32 -0700 Subject: 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 --- go/quickstep/src/com/android/quickstep/TaskListLoader.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'go') 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 tasks = new ArrayList<>(loadedTasks); // Reverse tasks to put most recent at the bottom of the view Collections.reverse(tasks); // Load task content -- cgit v1.2.3