diff options
| author | Hugo Benichi <hugobenichi@google.com> | 2017-07-25 21:57:51 +0900 |
|---|---|---|
| committer | Hugo Benichi <hugobenichi@google.com> | 2017-07-27 21:20:20 +0900 |
| commit | 5fcd050e0ecd5985cf184f55ea3df4434da8f824 (patch) | |
| tree | 25d4ce128637684578e917d5891b6f6f9e205562 /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | f44ebf5fcba4dc5f7021a3b72a8f26b11ae23db2 (diff) | |
NetworkNotificationManager: correctly handle existing notifications
This patch corrects a regression added by commit fb2609d3eee1 that did
not take into account the case of multiple notifications shown for a
single network id. Given how network notifications are triggered, it can
happen that NO_INTERNET and SIGN_IN notifications are both triggered for
the same network when captive portal detection is slow.
Contrary to the situation before commit fb2609d3eee1, a notification
priority order is introduced so that SIGN_IN always overrides
NO_INTERNET, and NO_INTERNET is ignored if SIGN_IN is already present.
Bug: 63676954
Bug: 62503737
Test: runtest frameworks-net, added new unit tests
Change-Id: Ib8658601e8d4dc6c41b335ab7dd8caa0cccd9531
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions