Chromium Code Reviews
DescriptionFullscreen Notifications: Fix crash if delegate is NULL
Notification Delegates should never be NULL but it is not enforced with
a DCHECK, so of course there is a delegate that is null (for the ChromeOS
Battery notification). So fix the logic that never checks the delegate's
validity in FullscreenNotificationBlocker.
R=bmalcolm@chromium.org
BUG=660889
TEST=have a fullscreen app window and drain the battery
below the warning level.
Review-Url: https://codereview.chromium.org/2459263004
Cr-Commit-Position: refs/heads/master@{#428825}
(cherry picked from commit b58f87810fac6a4d5ef606032a35ed8e8b179bd2)
Committed: https://chromium.googlesource.com/chromium/src/+/3bd6317c8d9449690991c07ff4f1957449d70fee
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||