404 artwork should reset notification artwork |
|||
Issue descriptionChrome Canary (57.0.2960.0) What steps will reproduce the problem? (1) Make sure artwork URL is valid and set metadata artwork (2) Notification is updated with new artwork (3) Make sure artwork URL is NOT valid (404 for instance) and set metadata artwork What is the expected result? Notification is updated and doesn't contain any artwork. What happens instead? Notification still contains old artwork
,
Jan 3 2017
I believe this is the same then. See https://beaufortfrancois.github.io/sandbox/media-session/404-artwork.html
,
Jan 3 2017
,
Jan 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/994960d632298a60bb6974a3b0a0217023eef4f0 commit 994960d632298a60bb6974a3b0a0217023eef4f0 Author: zqzhang <zqzhang@chromium.org> Date: Thu Jan 05 12:48:00 2017 [MediaNotification] Fix logic for resetting MediaImage This CL fixes the logic for unsetting MediaImage in the notification in the following two cases: * When image download returns 404. * When setting null metadata. BUG= 676997 R=mlamouri@chromium.org Review-Url: https://codereview.chromium.org/2606363002 Cr-Commit-Position: refs/heads/master@{#441636} [modify] https://crrev.com/994960d632298a60bb6974a3b0a0217023eef4f0/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaImageManager.java [modify] https://crrev.com/994960d632298a60bb6974a3b0a0217023eef4f0/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java [modify] https://crrev.com/994960d632298a60bb6974a3b0a0217023eef4f0/chrome/android/java_sources.gni [add] https://crrev.com/994960d632298a60bb6974a3b0a0217023eef4f0/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaImageManagerTest.java
,
Jan 5 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by zqzh...@chromium.org
, Jan 3 2017Labels: OS-Android
Owner: zqzh...@chromium.org
Status: Unconfirmed (was: Untriaged)