ef6721840337 - platform/frameworks/base

commit	ef6721840337bbff520d033fcae4122d4d34d3d4	[log] [tgz]
author	Jeff DeCew <jeffdq@google.com>	Thu May 05 15:06:17 2022 +0000
committer	Jeff DeCew <jeffdq@google.com>	Fri Jun 24 17:08:59 2022 +0000
tree	97e242da3feedc04e3c1d17db4e275fda83b6fb9
parent	5cd0e659310d7308df6fc501c6b379787bdfbeee [diff]

When applying a RankingMap that lacks an entry, remove that entry from the NotifCollection.

This fixes a bug after notification removed messages were dropped during moments of mass removal.  The NotifCollection was detecting this edge case but ignoring it.  Now we'll actually remove the notification and require an add event to get it back.

Note: this feature is still flagged off by default.

Bug: 216384850
Test: atest NotifCollectionTest
Change-Id: Id897c8761e40fd1adedd7c9e5a7bd2f9467d13ed
Merged-In: Id897c8761e40fd1adedd7c9e5a7bd2f9467d13ed

packages/SystemUI/src/com/android/systemui/flags/Flags.java[diff]
packages/SystemUI/src/com/android/systemui/statusbar/notification/NotifPipelineFlags.kt[diff]
packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/NotifCollection.java[diff]
packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/collection/NoManSimulator.java[diff]
packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/collection/NotifCollectionTest.java[diff]

5 files changed