Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 22f2f076 authored by Mady Mellor's avatar Mady Mellor
Browse files

When summary is dismissed, don't remove bubble children

Adds to onNotificationRemoveRequested to factor in:
  * summary dismissal: in this case the remove isn't intercepted, however, any bubble children in
    the children of the group are updated to not be visible in the shade
  * summary cancel: in this case if the summary is canceled the bubble
    children are also canceled
  * REASON_GROUP_SUMMARY_CANCELED - this is the indicator that the child
    is being removed because the summary was dismissed, in this case treat
    the bubble like its notification has been dismissed

Adds to NotificationManagerService so that NoMan won't remove any notifs
that are FLAG_BUBBLE because of a group summary removal.

Test: atest NotificationManagerServiceTest

Manual Tests:
1) User Dismiss
   - add enough bubbles to auto-bundle the notifications
   - swipe away the summary
   => Note that the bubbles still exist but now the dot is gone
2) Clear all (repeat above but clear all rather than swipe away)
3) Click on group
   - add a group with auto-cancel enabled
   - click on the group summary
   => Bubbles from that group should remain but the dot should be gone

Bug: 130250809
Change-Id: I2728152665817a0166cee72d969cbfdabc47067d
parent f1d559ef
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment