commit 54bbbdfa612be35990a9a5bcdd687825fac5c7b6
author Sunny Goyal <sunnygoyal@google.com> Wed Aug 07 13:49:41 2019 +0000
committer Sunny Goyal <sunnygoyal@google.com> Wed Aug 07 13:49:41 2019 +0000
tree 5252d3f5737c66ec68f30e62fc24a87429c46d5d
parent a04997b0811f2af79c0a91046d85d7e992496cab

Revert "Use new onTaskListUpdated() callback"

This reverts commit a04997b0811f2af79c0a91046d85d7e992496cab.

Reason for revert: Platform master has API changes, and launcher-master should still map to qpr branch

Change-Id: Ib201c71b5557715d585d322ec2a1a528e352e61c

quickstep/src/com/android/quickstep/RecentTasksList.java

diff --git a/quickstep/src/com/android/quickstep/RecentTasksList.java b/quickstep/src/com/android/quickstep/RecentTasksList.java
index 9b647b8..71ce32b 100644
--- a/quickstep/src/com/android/quickstep/RecentTasksList.java
+++ b/quickstep/src/com/android/quickstep/RecentTasksList.java
@@ -119,8 +119,12 @@
     }
 
     @Override
-    public void onRecentTaskListUpdated() {
+    public synchronized void onTaskStackChanged() {
         mChangeId++;
+    }
+
+    @Override
+    public void onTaskRemoved(int taskId) {
         mTasks = loadTasksInBackground(Integer.MAX_VALUE, false);
     }
 

quickstep/src/com/android/quickstep/RecentsModel.java

diff --git a/quickstep/src/com/android/quickstep/RecentsModel.java b/quickstep/src/com/android/quickstep/RecentsModel.java
index ede56b6..650169c 100644
--- a/quickstep/src/com/android/quickstep/RecentsModel.java
+++ b/quickstep/src/com/android/quickstep/RecentsModel.java
@@ -16,12 +16,15 @@
 package com.android.quickstep;
 
 import static com.android.quickstep.TaskUtils.checkCurrentOrManagedUserId;
+import static com.android.systemui.shared.system.QuickStepContract.KEY_EXTRA_SUPPORTS_WINDOW_CORNERS;
+import static com.android.systemui.shared.system.QuickStepContract.KEY_EXTRA_WINDOW_CORNER_RADIUS;
 
 import android.annotation.TargetApi;
 import android.app.ActivityManager;
 import android.content.ComponentCallbacks2;
 import android.content.Context;
 import android.os.Build;
+import android.os.Bundle;
 import android.os.HandlerThread;
 import android.os.Process;
 import android.os.RemoteException;
@@ -33,11 +36,12 @@
 import com.android.systemui.shared.recents.model.Task;
 import com.android.systemui.shared.recents.model.ThumbnailData;
 import com.android.systemui.shared.system.ActivityManagerWrapper;
+import com.android.systemui.shared.system.BackgroundExecutor;
 import com.android.systemui.shared.system.KeyguardManagerCompat;
+import com.android.systemui.shared.system.QuickStepContract;
 import com.android.systemui.shared.system.TaskStackChangeListener;
 
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.List;
 import java.util.function.Consumer;
 
@@ -128,7 +132,7 @@
     }
 
     @Override
-    public void onRecentTaskListUpdated() {
+    public void onTaskStackChangedBackground() {
         if (!mThumbnailCache.isPreloadingEnabled()) {
             // Skip if we aren't preloading
             return;
@@ -143,11 +147,7 @@
         // Keep the cache up to date with the latest thumbnails
         int runningTaskId = RecentsModel.getRunningTaskId();
         mTaskList.getTaskKeys(mThumbnailCache.getCacheSize(), tasks -> {
-            Collection<Task.TaskKey> currentKeys = mThumbnailCache.getTaskKeys();
-            List<Task.TaskKey> newKeys = new ArrayList<>(currentKeys.size());
-
             for (Task task : tasks) {
-                newKeys.add(task.key);
                 if (task.key.id == runningTaskId) {
                     // Skip the running task, it's not going to have an up-to-date snapshot by the
                     // time the user next enters overview
@@ -155,13 +155,6 @@
                 }
                 mThumbnailCache.updateThumbnailInCache(task);
             }
-
-            // Remove all keys we had before but no longer are in recents now
-            currentKeys.removeAll(newKeys);
-            for (Task.TaskKey tk : currentKeys) {
-                mThumbnailCache.remove(tk);
-            }
-
         });
     }
 
@@ -177,6 +170,12 @@
         }
     }
 
+    @Override
+    public void onTaskRemoved(int taskId) {
+        Task.TaskKey dummyKey = new Task.TaskKey(taskId, 0, null, null, 0, 0);
+        mThumbnailCache.remove(dummyKey);
+    }
+
     public void setSystemUiProxy(ISystemUiProxy systemUiProxy) {
         mSystemUiProxy = systemUiProxy;
     }

quickstep/src/com/android/quickstep/TaskThumbnailCache.java

diff --git a/quickstep/src/com/android/quickstep/TaskThumbnailCache.java b/quickstep/src/com/android/quickstep/TaskThumbnailCache.java
index c7c44ae..57c5a27 100644
--- a/quickstep/src/com/android/quickstep/TaskThumbnailCache.java
+++ b/quickstep/src/com/android/quickstep/TaskThumbnailCache.java
@@ -31,7 +31,6 @@
 import com.android.systemui.shared.recents.model.ThumbnailData;
 import com.android.systemui.shared.system.ActivityManagerWrapper;
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.function.Consumer;
 
 public class TaskThumbnailCache {
@@ -180,10 +179,6 @@
         return request;
     }
 
-    public Collection<TaskKey> getTaskKeys() {
-        return mCache.getValues();
-    }
-
     /**
      * Clears the cache.
      */