diff options
| author | 2025-03-17 18:40:27 +0000 | |
|---|---|---|
| committer | 2025-03-18 15:11:41 +0000 | |
| commit | de64e37fe7931dbdc7cb7f31649f2c45f4e38f31 (patch) | |
| tree | 288171cef0fe62bcccba03cf516f3e98f11775bc /libs/androidfw/misc.cpp | |
| parent | fcf9d0864a50bce3c62dda82fe67dda57b723561 (diff) | |
[SB][Notif] Have notif chips be isHidden=true if app is visible.
Previously, if a notification's app was visible, we changed its model to
be OngoingActivityChipModel.Inactive. But there's 2 reasons I'd rather
model it as OngoingActivityChipModel.Active(isHidden=true):
1) For UiEvent metric logging, it's better to keep the chip as Active
the entire time so that we don't log a "chip removed" event whenever
the notification's app is open.
2) When we bring return animations to notification chips (b/202516970),
we'll need to make this update anyway.
Bug: 202516970
Bug: 386821418
Fixes: 396636550
Bug: 364653005
Flag: com.android.systemui.status_bar_notification_chips
Test: Have status bar notification chip, then open app -> verify chip
still disappears. Close app -> verify chip still re-appears.
Test: atest StatusBarNotificationChipsInteractorTest
NotifChipsViewModelTest
Change-Id: I87af8943acc82989fb54c11660f222adcb49da0b
Diffstat (limited to 'libs/androidfw/misc.cpp')
0 files changed, 0 insertions, 0 deletions