Show artwork on wearable and lockscreen if the page specify one |
||||||
Issue descriptionCurrently we don't use large icon on Android lockscreen and wearable because we used the favicon as large icon and the resolution is too low. After we introduced the MediaSession API, the page can now specify artwork of higher resolution so we can use the artwork in lockscreen and wearable if the page specifies one.
,
Jan 9 2017
Issue 679356 has been merged into this issue.
,
Jan 9 2017
Also the image will be used as lockscreen, since the image for lockscreen and Wearable background use the same API in the Android Support Library.
,
Jan 9 2017
,
Jan 10 2017
I thought it should already work with setLargeIcon. Am I wrong? https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java?q=setVisibility+media+android&sq=package:chromium&dr=C&l=857
,
Jan 10 2017
No, setLargeIcon only for the notification icon. For lockscreen and Wearable background, we need to use the Android MediaSessionCompat in the support library: 1. Construct a MediaMetadataCompat with METADATA_KEY_ALBUM_ART. 2. Call MediaSessionCompat.setMetadata() with the metadata in step 1.
,
Jan 12 2017
,
Jan 12 2017
,
Jan 12 2017
,
Jan 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5f9b3d7676beeb194091dae650ada4ee1a59f13e commit 5f9b3d7676beeb194091dae650ada4ee1a59f13e Author: zqzhang <zqzhang@chromium.org> Date: Sat Jan 14 17:11:59 2017 [Media>UI] Set large icon for lockscreen and wearable if it is specified by the page With MediaSession API, the page can specify custom media artwork. The quality of these images are usually good enough to be used as lockscreen and wearable background. This CL enables this usage. BUG= 678207 Review-Url: https://codereview.chromium.org/2626213004 Cr-Commit-Position: refs/heads/master@{#443800} [modify] https://crrev.com/5f9b3d7676beeb194091dae650ada4ee1a59f13e/chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java [modify] https://crrev.com/5f9b3d7676beeb194091dae650ada4ee1a59f13e/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java [modify] https://crrev.com/5f9b3d7676beeb194091dae650ada4ee1a59f13e/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationInfo.java [modify] https://crrev.com/5f9b3d7676beeb194091dae650ada4ee1a59f13e/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java [modify] https://crrev.com/5f9b3d7676beeb194091dae650ada4ee1a59f13e/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java
,
Jan 16 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by fbeaufort@chromium.org
, Jan 4 201723.2 KB
23.2 KB View Download