summaryrefslogtreecommitdiff
path: root/packages/Shell/src
diff options
context:
space:
mode:
author Caitlin Shkuratov <caitlinshk@google.com> 2024-11-08 22:11:16 +0000
committer Caitlin Shkuratov <caitlinshk@google.com> 2024-11-11 20:35:52 +0000
commite0b1ad1c1f8d999b5a7f28f8c4e2b8e4ac88e483 (patch)
tree071eed9d5260f9ef42badcaa61e814843f23c647 /packages/Shell/src
parent622356e8b32495038eb9feeafc857e5109472229 (diff)
[Notif] Merge HeadsUpManagerPhone into BaseHeadsUpManager.
Arc used to have its own implementation of BaseHeadsUpManager, which is why HeadsUpManagerPhone was originally created. Since the Arc implementation no longer exists, we can combine BaseHeadsUpManager and HeadsUpManagerPhone into a single class to reduce cognitive load. Also adds explicit @Override to methods that are implementing the HeadsUpManager interface, and moves Javadoc from HeadsUpManagerPhone to the HeadsUpManagerinterface. Otherwise, tries to leave the existing code unchanged. Bug: 378142453 Flag: EXEMPT refactor Test: SysUI + all variants compile Test: Key notification CUJs work: - Receive notif while unlocked -> notif HUNs - Swipe up on HUN -> notif snoozed - Swipe left/right on HUN -> notif dismissed - Tap on HUN -> launches activity - Receive notif while locked with bypass enabled -> notif HUNs - Receive notif while locked with bypass disabled -> notif becomes part of stack - Enable notification cooldown -> notifs cool down Test: atest BaseHeadsUpManagerTest HeadsUpManagerPhoneTest Change-Id: Id1722d845c910f95e9ec37c39f67195ddbf7226f
Diffstat (limited to 'packages/Shell/src')
0 files changed, 0 insertions, 0 deletions