commit | 3476cb25a4239c6877beb369865985855339c2b8 | [log] [tgz] |
---|---|---|
author | Tetsui Ohkubo <tetsui@chromium.org> | Fri Jan 11 01:34:25 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Jan 11 01:34:25 2019 |
tree | 7aee4d7f5ab01339e70fad621f95391b9b37af9e | |
parent | 0b15d057e7dacd7e14d2ff3b0308c1cc2b706695 [diff] |
NotificationViewMD: Validate after ToggleExpanded NotificationViewMD::ButtonPressed() crashes when a notification popup is expanded and that causes the popup to be removed. (It happens when there's not enough screen space and the topmost notification is expanded.) This CL checks if |this| is still valid after ToggleExpanded by using WeakPtr. Also it removes old workaround for the same issue that wasn't working. (Added in https://crrev.com/114503004) TEST=NotificationViewMD.TestDeleteOnToggleExpanded BUG=918933 Change-Id: I9e1290112d6eaa2ab7155eb291764cd38e854f7e Reviewed-on: https://chromium-review.googlesource.com/c/1400431 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#621850}
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 .