Commit 341c496b authored Jun 07, 2023 by Julia Tuttle Committed by Brian Egizi Jun 13, 2023

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

parent 6c50d5ae

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java

+0 −4

Original line number	Diff line number	Diff line
		@@ -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<NotificationEntry> entriesToLocallyDismiss = new ArrayList<>();
		for (int i = 0; i < entriesToDismiss.size(); i++) {
		NotificationEntry entry = entriesToDismiss.get(i).first;