Chromium Code Reviews
DescriptionHide the download summary on file open
Notifications can be dismissed with a
DownloadManager.ACTION_NOTIFICATION_CLICKED intent, which bypasses the
DownloadNotificationService so we don't get a chance to hide the
summary. This adds a check to the
DownloadBroadcastReceiver#openDownload() path that allows us to hide any
dangling summary notifications that shouldn't be visible. This also
lets us catch any clicks on the summary which gives us a chance to close
them out as well.
BUG=699323
Review-Url: https://codereview.chromium.org/2796573002
Cr-Commit-Position: refs/heads/master@{#461744}
(cherry picked from commit 803fdf80f842803c7c91dca2a87419afe1b3d60c)
Review-Url: https://codereview.chromium.org/2794353003 .
Cr-Commit-Position: refs/branch-heads/3029@{#578}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/f1549937850de9b1959a59b6f0e84a33c5319faf
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||||||||||||||||||||