Commit fe799ef0 authored by ojan@chromium.org's avatar ojan@chromium.org
Browse files

2010-01-06 Ojan Vafai <ojan@chromium.org>

        Reviewed by Eric Carlson.

        media/video-seeking.html is flaky
        https://bugs.webkit.org/show_bug.cgi?id=33273

        Removed setTimeouts from this test. It kind of looks
        like they were intentional, but I don't see how they add
        anything to this test (other than race conditions).

        * media/video-seeking-expected.txt:
        * media/video-seeking.html:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@52881 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 80b9507c
2010-01-06 Ojan Vafai <ojan@chromium.org>
Reviewed by Eric Carlson.
media/video-seeking.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=33273
Removed setTimeouts from this test. It kind of looks
like they were intentional, but I don't see how they add
anything to this test (other than race conditions).
* media/video-seeking-expected.txt:
* media/video-seeking.html:
2010-01-06 Ojan Vafai <ojan@chromium.org>
 
Reviewed by Eric Seidel.
......@@ -5,7 +5,6 @@ RUN(video.currentTime = 0.5)
EVENT(seeked)
EXPECTED (seekedCount == '1') OK
EXPECTED (video.seeking == 'false') OK
EXPECTED (video.currentTime == '0.5') OK
RUN(video.currentTime = 1)
......@@ -13,7 +12,6 @@ EXPECTED (video.seeking == 'true') OK
EVENT(seeked)
EXPECTED (seekedCount == '2') OK
EXPECTED (video.seeking == 'false') OK
EXPECTED (video.currentTime == '1') OK
RUN(video.currentTime = 1.5)
......@@ -21,7 +19,6 @@ EXPECTED (video.seeking == 'true') OK
EVENT(seeked)
EXPECTED (seekedCount == '3') OK
EXPECTED (video.seeking == 'false') OK
EXPECTED (video.currentTime == '1.5') OK
RUN(video.currentTime = 2)
......
......@@ -8,39 +8,26 @@
<script>
var seekedCount = 0;
var counter;
function seeked()
{
++seekedCount;
consoleWrite("");
}
function someTimeLater()
{
testExpected("seekedCount", counter);
testExpected("video.seeking", false);
testExpected("video.currentTime", counter * 0.5);
run("video.currentTime = " + (counter + 1) * 0.5);
testExpected("video.currentTime", seekedCount * 0.5);
run("video.currentTime = " + (seekedCount + 1) * 0.5);
testExpected("video.seeking", true);
if (counter == 3)
{
consoleWrite("");
endTest();
}
++counter;
consoleWrite("");
if (seekedCount == 3)
endTest();
}
function canplaythrough()
{
run("video.currentTime = 0.5");
counter = 1;
setTimeout(someTimeLater, 200)
setTimeout(someTimeLater, 400)
setTimeout(someTimeLater, 600)
consoleWrite("");
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment