From 4be13eb3112100b09f1ded51dc8f241d52b38e8a Mon Sep 17 00:00:00 2001
From: Ajinkya Chalke <achalke@google.com>
Date: Thu, 29 May 2025 08:01:51 -0700
Subject: [PATCH] Remove obsolete TODO

- From b/399089118#comment2 "On external dispalys, RecentsViewContainer is always RecentsWindowManager, so onOverviewTargetChanged will never be called on external displays". So we we only need to track OverviewTargetChange for default display.

Fix: 399089118
Test: m
Flag: EXEMPT comment only change
Change-Id: Ica0527e45da6f0079bae5b69e9100b9f74d7f7ad
---
 .../src/com/android/quickstep/TouchInteractionService.java      | 2 --
 1 file changed, 2 deletions(-)

diff --git a/quickstep/src/com/android/quickstep/TouchInteractionService.java b/quickstep/src/com/android/quickstep/TouchInteractionService.java
index 4033ebe3ba..b3ce9414cd 100644
--- a/quickstep/src/com/android/quickstep/TouchInteractionService.java
+++ b/quickstep/src/com/android/quickstep/TouchInteractionService.java
@@ -908,8 +908,6 @@ public class TouchInteractionService extends Service {
 
     private void onOverviewTargetChanged(boolean isHomeAndOverviewSame) {
         mAllAppsActionManager.setHomeAndOverviewSame(isHomeAndOverviewSame);
-        // TODO (b/399089118): how will this work with per-display Taskbars? Is using the
-        //  default-display container ok?
         RecentsViewContainer newOverviewContainer =
                 mOverviewComponentObserver.getContainerInterface(
                         DEFAULT_DISPLAY).getCreatedContainer();