Comment #85 on issue 13907 by [email protected]: Layout tests failures
for HTML5 audio/video
http://code.google.com/p/chromium/issues/detail?id=13907
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=29851
------------------------------------------------------------------------
r29851 | [email protected] | 2009-10-22 17:56:24 -0700 (Thu, 22 Oct
2009) | 11 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.cc?r1=29851&r2=29850
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.h?r1=29851&r2=29850
Use a saved copy of the time when pausing media elements.
Since our internal pause is asynchronous and we don't want to hang the
render thread, it's possible to have our clock creep ahead. This produces
unexpected behaviour when pausing media elements, such as seeing
currentTime attribute nudge a little ahead after calling pause().
This so happens to also address layout test flakiness.
BUG=13907
TEST=layout tests, especially video-pause-immediately.html -- a new layout
test I wrote and am upstreaming
Review URL: http://codereview.chromium.org/334002
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
