New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 680058 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Android MediaRouter only (left Chro...
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

play & pause MediaSessionActionHandlers are not triggered

Project Member Reported by fbeaufort@chromium.org, Jan 11 2017

Issue description

Chromium (57.0.2979.0)

What steps will reproduce the problem?
1. Go to https://beaufortfrancois.github.io/sandbox/media-session/all-media-session-action-handlers.html
2. Play audio file
3. Tap all media controls from left to right

What is the expected result?
Web page should show:
 
previoustrack
seekbackward
pause
play
seekforward
nexttrack


What happens instead of that?

Web page shows:
 
previoustrack
seekbackward
seekforward
nexttrack


For some reason, play and pause media session action handlers are not triggered.

 
Owner: zqzh...@chromium.org
Cc: mlamouri@chromium.org
This is because currently we handle play/pause in Chrome internally and do not send them to the page. Ideally it's not good, but we didn't violate the spec. maybe we can print a console warning temporarily to let developers know what's going on
Labels: -Pri-2 Pri-3
Labels: -Pri-3 Pri-1
Labels: M-57
Status: Started (was: Unconfirmed)
Project Member

Comment 8 by bugdroid1@chromium.org, Jan 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e6e31527e9ace8760b0b89f738098d8dbf6d12cb

commit e6e31527e9ace8760b0b89f738098d8dbf6d12cb
Author: zqzhang <zqzhang@chromium.org>
Date: Wed Jan 18 12:00:10 2017

[MediaSession] Send play/pause action to the page if the handler is set

Previously we handle play/pause actions internally and do not send them
to the page. We should let the page handle them if it specifies handlers
for play/pause.

BUG= 680058 

Review-Url: https://codereview.chromium.org/2621333002
Cr-Commit-Position: refs/heads/master@{#444337}

[modify] https://crrev.com/e6e31527e9ace8760b0b89f738098d8dbf6d12cb/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java

Status: Fixed (was: Started)
I guess this is fixed.
Verified in Chromium (57.0.2987.0)

Sign in to add a comment