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

Issue 616772 link

Starred by 1 user
Status: Verified
Owner:
█ ti...@chromium.org
Email to this user bounced
Closed: Jun 2016
Cc:
qin...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

pause doesn't work for an audio element

Reported by nederlan...@gmail.com, Jun 2 2016 
Example URL:
https://www.blend4web.com/media/uploads/4fdb0405-3ee9-4781-a825-ef00252a2406/test.html

Steps to reproduce the problem:
1. create an HTML page with an audio element with a valid src and "loop" and "controls" attributes set
2. press play
3. press pause

What is the expected behavior?
The audio element stops playing the sound

What went wrong?
The audio element is still playing the sound, although the controls indicate that it's stopped.

Did this work before? N/A 

Is it a problem with Flash or HTML5? HTML5

Does this work in other browsers? Yes 

Chrome version: 50.0.2661.89  Channel: stable
OS Version: 6.0.1
Flash Version: Shockwave Flash 22.0 r0

This is reproduced on android 6.0.1 (Nexus 6 and Nexus 9 device) in stable 50.0.2661.89 and dev 51.0.2704.77.

This isn't reproduced on android 5.0 (Samsung Galaxy Note4 and YotaPhone 2) on exactly the same versions.
test.html
150 bytes View Download

Comment 1 by krav...@chromium.org, Jun 3 2016

Labels: M-52
Owner: qin...@chromium.org
Status: Assigned (was: Unconfirmed)
able to repro on Nexus6

Comment 2 by dalecur...@chromium.org, Jun 3 2016

Cc: ti...@chromium.org
Should be old media player, but can you confirm that there are no entries in chrome://media-internals when you try this?
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 4 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by nederlan...@gmail.com, Jun 6 2016 

> Should be old media player, but can you confirm that there are no entries in chrome://media-internals when you try this?

Yes, chrome://media-internals just displays "none" under "Players".

Comment 5 by ti...@chromium.org, Jun 8 2016 

Reproducible with old pipeline with Nexus 6P, angler-userdebug N NRD67 2943831 dev-keys and Clankium 53.0.2762.0.
Spitzer does not have this problem.

Comment 6 by ti...@chromium.org, Jun 8 2016 

It looks like if "loop" is set WMPA::OnPlaybackComplete() can be called after Pause() and that will cause restart.

06-08 11:40:49.672 30521 30536 V timav   : [30521/30536] 18:40:49.672 [webmediaplayer_android.cc:454] pause
06-08 11:40:49.672 30521 30536 V timav   : [30521/30536] 18:40:49.672 [webmediaplayer_android.cc:1186] Pause: is_media_related_action:1
06-08 11:40:49.676 30257 30257 V timav   : [30257/30257] 11:40:49.676 [media_player_bridge.cc:325] Pause is_media_related_action:1
06-08 11:40:49.676 30257 30257 V timav   : [30257/30257] 11:40:49.676 [media_player_bridge.cc:539] PauseInternal
06-08 11:40:49.676   579  3075 D NuPlayerDriver: pause(0xf480c0c0)
06-08 11:40:49.676   579  3075 D NuPlayerDriver: notifyListener_l(0xf480c0c0), (7, 0, 0), loop setting(0, 0)
06-08 11:40:49.864   579 30566 D NuPlayerDriver: notifyListener_l(0xf480c0c0), (2, 0, 0), loop setting(0, 0)
06-08 11:40:49.866 30521 30536 V timav   : [30521/30536] 18:40:49.866 [webmediaplayer_android.cc:865] OnPlaybackComplete
06-08 11:40:49.866 30521 30536 V timav   : [30521/30536] 18:40:49.866 [webmediaplayer_android.cc:469] seek(0)
06-08 11:40:49.867 30521 30536 V timav   : [30521/30536] 18:40:49.867 [webmediaplayer_android.cc:878] OnPlaybackComplete: calling player_manager_->Start()
06-08 11:40:49.868   579   717 D NuPlayerDriver: seekTo(0xf480c0c0) 0 ms at state 6
06-08 11:40:49.868   579   717 D NuPlayerDriver: notifyListener_l(0xf480c0c0), (7, 0, 0), loop setting(0, 0)
06-08 11:40:49.868 30257 30257 V timav   : [30257/30257] 11:40:49.868 [media_player_bridge.cc:301] Start
06-08 11:40:49.869 30257 30257 V timav   : [30257/30257] 11:40:49.868 [media_player_bridge.cc:522] StartInternal
06-08 11:40:49.871   937  2963 I MediaFocusControl:  AudioFocus  requestAudioFocus() from uid/pid 10091/30257 clientId=android.media.AudioManager@f0f40cforg.chromium.content.browser.MediaSessionDelegate@e57da5c req=3 flags=0x0
06-08 11:40:49.872   579 30566 D NuPlayerDriver: notifyListener_l(0xf480c0c0), (4, 0, 0), loop setting(0, 0)
06-08 11:40:49.872   579 30566 D NuPlayerDriver: notifyListener_l(0xf480c0c0), (3, 100, 0), loop setting(0, 0)
06-08 11:40:49.873   579  3075 D NuPlayerDriver: start(0xf480c0c0), state is 6, eos is 0

Comment 7 by ti...@chromium.org, Jun 8 2016

Cc: -ti...@chromium.org qin...@chromium.org
Owner: ti...@chromium.org
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 8 2016 

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

commit 8b8ca53230694ddbaa23efa1042c9f887058e88c
Author: timav <timav@chromium.org>
Date: Wed Jun 08 22:48:31 2016

Prevent playback looping after pause.

Check is_playing_ vatiable in WMPA::OnPlaybackComplete
before restarting the player.

BUG= 616772 

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

[modify] https://crrev.com/8b8ca53230694ddbaa23efa1042c9f887058e88c/content/renderer/media/android/webmediaplayer_android.cc

Comment 9 by ti...@chromium.org, Jun 13 2016

Status: Fixed (was: Assigned)

Comment 10 by krav...@chromium.org, Jun 14 2016

Status: Verified (was: Fixed)
Verified in 53.0.2767.3 build
►

Sign in to add a comment