diff options
| author | Jeff DeCew <jeffdq@google.com> | 2022-01-14 01:52:13 +0000 |
|---|---|---|
| committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2022-01-28 01:20:07 +0000 |
| commit | b479d6487827d5ee39f58d76b1b5727b57ea02d1 (patch) | |
| tree | 759f15487325bba1a88fee89533752eb85da4117 /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | 878e9322034d9de4f4b87a61b24e4ba4af908d5f (diff) | |
Allow alertOverride if the summary GROUP_ALERT_ALL
Because we still require all children to have GROUP_ALERT_PARENT, the meaning of GROUP_ALERT_ALL is equivalent to GROUP_ALERT_PARENT in any case where we would have an alertOverride, so we now allow either of those values.
Also of note, because the alertOverride absolutely needs to have GROUP_ALERT_PARENT, and the semantics of heterogeneous child alert behaviors is unclear, we're keeping the requirement that all children have GROUP_APERT_PARENT to avoid over-applying this logic.
Bug: 185680162
Test: atest NotificationGroupManagerLegacyTest
Merged-In: I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
Change-Id: I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
(cherry picked from commit 6ee64f6f263f183b27f7b24d30d74bce1ceef660)
(cherry picked from commit 99b1011efdeb73dc15ded2e6b1cd569b4ed07373)
Merged-In:I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions