commit | 13330ec99e9b599a060b6808075ebaaec139569e | [log] [tgz] |
---|---|---|
author | Tim Song <tengs@chromium.org> | Tue Mar 19 15:28:51 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Mar 19 15:28:51 2019 |
tree | a0e97645a8113d12c49c3696f89b32e8aa9801a6 | |
parent | 18bafcb1b62cdd3dbcbb68dacd1927bfbee506ee [diff] |
Notifications: Allow removing non-visible notifications in MessageCenter. Currently, MessageCenter::RemoveNotification() only removes "visible" notifications. In multi-user mode (and other scenarios), notifications for non-active users are blocked and not "visible". When Android dismisses an ARC notification from a non-active user, the corresponding MessageCenter notification is not properly removed. Accessing the invalid notification will result in a crash. BUG=940060 TEST=new unit test, verified bug does not repro Change-Id: Ic6688b4d7137777d7b77dc07a823681927975dc8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1530476 Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tim Song <tengs@chromium.org> Cr-Commit-Position: refs/heads/master@{#641913}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .