commit ae2c01a0b67b2daf424c7caf62b4f703a79d5789
author Uwais Ashraf <uwaisashraf@google.com> Tue Oct 29 18:39:34 2024 +0000
committer Uwais Ashraf <uwaisashraf@google.com> Tue Nov 05 09:22:37 2024 +0000
tree 11a96b5eb422ac61e7a7b1280f73619973615401
parent 972531f52994b2a62053e7c93574430eef5ef95e

Reduce log frequency of TasksRepository.setVisibleTasks()

Fix: 377438334
Bug: 377437341
Flag: com.android.launcher3.enable_refactor_task_thumbnail
Test: SwitchBackToSplitFromRecentGesturalNavLandscapeMicrobenchmark
Test: QuickSwitchTasksMicrobenchmark
Change-Id: I0cf56995ece9187a9a4da3ad61a17a795d0934db

quickstep/src/com/android/quickstep/recents/data/TasksRepository.kt

diff --git a/quickstep/src/com/android/quickstep/recents/data/TasksRepository.kt b/quickstep/src/com/android/quickstep/recents/data/TasksRepository.kt
index 4f38ec7..275af00 100644
--- a/quickstep/src/com/android/quickstep/recents/data/TasksRepository.kt
+++ b/quickstep/src/com/android/quickstep/recents/data/TasksRepository.kt
@@ -73,13 +73,17 @@
         getTaskDataById(taskId).map { it?.thumbnail }.distinctUntilChangedBy { it?.snapshotId }
 
     override fun setVisibleTasks(visibleTaskIdList: Set<Int>) {
-        Log.d(TAG, "setVisibleTasks: $visibleTaskIdList")
-
         // Remove tasks are no longer visible
         val tasksNoLongerVisible = taskRequests.keys.subtract(visibleTaskIdList)
         removeTasks(tasksNoLongerVisible)
         // Add new tasks to be requested
-        visibleTaskIdList.subtract(taskRequests.keys).forEach { taskId -> requestTaskData(taskId) }
+        val newlyVisibleTasks = visibleTaskIdList.subtract(taskRequests.keys)
+        newlyVisibleTasks.forEach { taskId -> requestTaskData(taskId) }
+
+        if (tasksNoLongerVisible.isNotEmpty() || newlyVisibleTasks.isNotEmpty()) {
+            Log.d(TAG, "setVisibleTasks to: $visibleTaskIdList, " +
+                    "removed: $tasksNoLongerVisible, added: $newlyVisibleTasks")
+        }
     }
 
     private fun requestTaskData(taskId: Int) {