All media session action buttons should be shown if all action handlers are set |
|||||||
Issue descriptionChrome Version : 63.0.3222.0 Android: 5, 6, 7, and 8 What steps will reproduce the problem? 1. Go to https://googlechrome.github.io/samples/media-session/video.html 2. Check out media session notification What is the expected result? I should see all media session action buttons (including Skip Next and Previous Track) What happens instead of that? I only see pause/play and seek backwards and seek forwards buttons on Android 5 and 6 while I can see all of them in Android 7 and 8. See screenshots below.
,
Oct 18 2017
Just to add more info about this issue. Tested the issue using #63.0.3222.0 on Android SM-J710F Build/MMB29K, 5.0.0:; Micromax A106 Build?LRX21M and observed the same behavior as mentioned in comment #0. Thanks!1
,
Oct 18 2017
I believe this is an OS restriction where the notifications are shown in compact view in 5.0/6.0.
,
Oct 19 2017
I don't think it's an OS restriction. Android 5.0 and 6.0 have a different layout when there are 5 actions. At the time, it was meant to unappealing to be used. Though, I think consistency is more important.
,
Oct 31 2017
,
Nov 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ac270dea6409d067be283c99fa1905a0e1f7e40f commit ac270dea6409d067be283c99fa1905a0e1f7e40f Author: Mounir Lamouri <mlamouri@chromium.org> Date: Wed Nov 01 11:49:30 2017 Media Notifications: allow to use 5 actions for pre-N devices. There was a hard coded limitation to 3 actions because the pre-N notifications do not look great with more than 3 actions but it is breaking consistency for websites using the Media Session API. This is also renaming `isRunningN()` to `isRunningAtLeastN()`. Bug: 775471 Change-Id: I2e9289ff0daa987acf0b656a239069676b03d722 Reviewed-on: https://chromium-review.googlesource.com/747221 Reviewed-by: Tommy Steimel <steimel@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#513107} [modify] https://crrev.com/ac270dea6409d067be283c99fa1905a0e1f7e40f/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
,
Nov 1 2017
,
Nov 1 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by ligim...@chromium.org
, Oct 17 2017