New issue
Advanced search Search tips

Issue 596173 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Investigate non-linear current time increases on Android (others?).

Project Member Reported by dalecur...@chromium.org, Mar 18 2016

Issue description

Split from  issue 263654 :

https://bugs.chromium.org/p/chromium/issues/detail?id=263654#c72
Hm, I had the wrong version written down, it was of course the 51 Dev version, 51.0.2681.4.

So the fix should be there?  In that case, it doesn't really seem to make a very large impact.  I've got a test on http://fanoli01.itek.norut.no/Analysis/test.html - which plots the progress of currentTime compared to performance.now() (normalized for playback speed 1).  In an ideal world, you'd see a horizontal line, Chrome 51 has changed from earlier versions of course.  You can see some versions (not so many on Android) on http://fanoli01.itek.norut.no/Analysis/results.html  (use a wide browser...)

Importantly, the plot uses currentTime as reported immediately after a timechange event.  This is what gives us better accuracy in other browsers. Perhaps it is not really fresh, or compensated for before the event is triggered?

The flatter the line, the better we can synchronize it.  Please feel free to use the test, or if it fails to provide sufficient information, don't hesitate to ask!

https://bugs.chromium.org/p/chromium/issues/detail?id=263654#c74
http://fanoli01.itek.norut.no/Analysis/test2.html uses requestAnimationFrame instead - even though running getAnimationFrame back-to-back swamps everything to bits.  I've set it on a timer (after 200ms getAnimationFrame is requested).  It looks quite similar for audio at least, not at all even.  The basic thing the test does is:

* on first currentTime update (or query), remember performance.now() as startTime
* on each subsequent update, calculate the diff as elem.currentTime - (performance.now() - startTime)/1000 

One would expect that elem.currentTime is perfectly linear to performance.now() if it is calculated on each request.  The plots I get are not.  I made the trivial one at http://fanoli01.itek.norut.no/test/ if you want to take a look at it.

 
@njaal.borch -- is your test using playback rate changes? I haven't looked into the code too deeply. 
No, this is with fixed playback rate. 
Project Member

Comment 3 by sheriffbot@chromium.org, Mar 20 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue.
The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: WontFix (was: Untriaged)
stale bug for > 1 year. close it.

Sign in to add a comment