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

Issue 622325 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

HTML5 loop broken both when using loop attribute and manual looping via callback functions

Reported by reinis.a...@gmail.com, Jun 22 2016

Issue description

Example URL:
http://jsbin.com/fijaxin/4/edit?html,js,output

Steps to reproduce the problem:
Check the urls provided, that has looping video
1. Loop via attribute
http://jsbin.com/fijaxin/4/edit?html,js,output
2. Loop via callbacks
http://jsbin.com/wuqexo/2/edit?html,js,output

What is the expected behavior?
Video and audio should loop infinitely

What went wrong?
Behaviour types observed:
1) first loop all fine, on second loop audio is gone, third loop never starts (both audio and video)
1) first loop all fine, starting second loop video gets jagged, audio & video out of sync till infinity
This is both when MP4 and WEBM formats are used.

Did this work before? N/A 

Is it a problem with Flash or HTML5? HTML5

Does this work in other browsers? Yes 

Chrome version: 51.0.2704.81  Channel: stable
OS Version: 5.1.1
Flash Version: 

Full user agent string:
Mozilla/5.0 (Linux; Android 5.1.1; SM-P605 Build/LMY47X; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/51.0.2704.81 Safari/537.36
 
Labels: M-52
Owner: qin...@chromium.org
Status: Assigned (was: Unconfirmed)
able to repro Nexus9/MOB30P/51.0.2704.81
Cc: qin...@chromium.org
Owner: dalecur...@chromium.org
Is this fixed if you go to chrome://flags and enable unified media pipeline?
(Can't repro with the unified media pipeline at least, M52)
Project Member

Comment 4 by sheriffbot@chromium.org, Jun 25 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
--enable-unified-media-pipeline fixes it for Chrome Browser.

apart from original questio which is solved I'd like to add that this affects Chromium WebView 51.0.2704.81 which in turn affects everybody that loads webview in non-debug-mode (which is way all kiosk browser apps that I have checked in the market do it) as 
adb shell 'echo "chrome --enable-unified-media-pipeline" > /data/local/tmp/webview-command-line'
will be ignored https://android.googlesource.com/platform/frameworks/webview/+/android-cts-5.1_r8/chromium/java/com/android/webview/chromium/WebViewChromiumFactoryProvider.java#117
if I understand the code right
Cc: dalecur...@chromium.org
Owner: ti...@chromium.org
timav@ did you fix something here w/ looping recently?

Comment 7 Deleted

As a desperate user who has tried everything lately (including Firefox, where looping works hehe) I can see that there are some changes in latest chrome beta for android 52.0.2743.49.
"--enable-unified-media-pipeline" (unified pipeline is OFF by default) has become "--disable-unified-media-pipeline" (and unified pipline is ON by default). Thus previously one had to set the flag and looping worked, in latest beta looping does not glitch by default.

Btw I posted related ticket today https://bugs.chromium.org/p/chromium/issues/detail?id=623943
As stated above in this thread for standalone Chrome Browser there is this ability for END USER (meaning somebody that does not know what rooting, adb tools and setting flags in COMMAND_LINE_FILE path is) to set flag (now default on), but that does not apply to straight-WebView-dependant things which I am actually looking for (Android app that uses built in WebWiew cannot loop video).
So... if the latest Chrome Browser 52.0.2743.49 beta by default uses unified-media-pipeline, I hope that it will make it's way someday also in WebView (52.0.2743.49 beta does not has it).

Thank you!
Yeah sorry, that was supposed to be default on for M51 but we had some trouble, we are aiming for an M52 launch now.
No prob, thanks for your work! May I ask also there then here quickly - is it planned for unified pipline to be by default ON for final WebView R52 (as mentioned earlier - currently Chrome Browser 52.0.2743.49 beta default is ON, WebWiev 52.0.2743.49 beta default is OFF)? I'm probably crossposting now, I don't know which devs here work on what, sorry for that. Say, and I will stop :) and just bump the ticket submitted in Mobile>WebView category time to time then. Thank you!
Not for WebView unfortunately, it won't be default on until M53 for WebView due to additional complexity in WebView.
Ok, thanks. So I'll wait starting July for first WebView 53.x betas that hopefully will have this and around mid September for R53 (according to Chromium Development Calendar ;)) Thank you again!
Project Member

Comment 14 by sheriffbot@chromium.org, Jul 3 2016

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

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

Comment 15 by ti...@chromium.org, Jul 14 2016

Cc: -dalecur...@chromium.org ti...@chromium.org
Owner: dalecur...@chromium.org
I'm unable to reproduce this issue neither on N9 with MOB30P (i.e. volantis-userdebug 6.0.1 MOB30P 2960889 dev-keys), system WebView 44.0.2403.117, nor on N9 with Android N build NRD90G.

On the other hand Spitzer was enabled for WebView in https://codereview.chromium.org/2057743002

I would mark it "fixed" if I could reproduce. Back to Dale.
Labels: M-53
Status: Fixed (was: Assigned)
Re-tested again and  unable to repro on Nexus 9 and Samsung Galaxy-SCH-I925/JZ054k

Sign in to add a comment