Chromium Code Reviews
DescriptionWorkaround for NotificationManager race condition
It looks like we occasionally handle the notification click intent
while the notification is still temporarily there. This check adds the
notification id of the dying notification to the intent, so we can
ignore it when scanning the existing notifications to see if any still
require us to have our summary notification active.
BUG=709560
Review-Url: https://codereview.chromium.org/2805843002
Cr-Commit-Position: refs/heads/master@{#463120}
(cherry picked from commit ce4c38c5b3535bcb4a08cf2771db378aa8f278f9)
Review-Url: https://codereview.chromium.org/2813723002 .
Cr-Commit-Position: refs/branch-heads/3029@{#649}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/e7b3984a7e4bf958d24085c50967156174a35329
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||||||||||||||||||||