From 341c496beaaf041c46a04c993a49ae1bcb5481e0 Mon Sep 17 00:00:00 2001 From: Julia Tuttle Date: Wed, 7 Jun 2023 17:13:55 -0400 Subject: [PATCH] Remove obsolete TODO b/206842750 was fixed but the TODO was never removed. Cherry picked I76e143150fedde851678a43c1909b1aefaca28bb to master due to b/286408867. Code was merged via `-s ours` and is missing even though the sha exists on the target branch. Cherry pick is required to bring in the code. Manually remove the "Merged in" directive to allow for downstream propagation. Bug: 206842750 Test: none (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:f2a8c30715c3af5938df0ab488732de694c89e2e) Change-Id: I1b05bf5749d3365510d6c8a9b20d2fe3eeffc1ff --- .../statusbar/notification/collection/NotifCollection.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java index 8aeefeeac2110..b116246586b57 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java @@ -274,10 +274,6 @@ public class NotifCollection implements Dumpable, PipelineDumpable { Assert.isMainThread(); checkForReentrantCall(); - // TODO (b/206842750): This method is called from (silent) clear all and non-clear all - // contexts and should be checking the NO_CLEAR flag, rather than depending on NSSL - // to pass in a properly filtered list of notifications - final List entriesToLocallyDismiss = new ArrayList<>(); for (int i = 0; i < entriesToDismiss.size(); i++) { NotificationEntry entry = entriesToDismiss.get(i).first;