Bug 88138 – [Gtk][EFL] http/tests/media/video-buffered.html is failing after r119268

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Zan Dobersek]

http/tests/media/video-buffered.html started failing after r119268[1].

Here's the diff:

--- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/http/tests/media/video-buffered-expected.txt 
+++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/http/tests/media/video-buffered-actual.txt 
@@ -1,9 +1,616 @@
+FAIL: Timed out waiting for notifyDone to be called
 Start playing a video with preloading enabled, do a seek near the end and check multiple buffered timeranges have been created.
 
 EVENT(playing)
 EXPECTED (video.buffered.length == '1') OK
 RUN(video.currentTime = video.duration - 0.5)
-EVENT(ended)
-EXPECTED (video.buffered.length == '2') OK
-END OF TEST
+EVENT(playing)
+EXPECTED (video.buffered.length == '1') OK
+RUN(video.currentTime = video.duration - 0.5)
+EVENT(playing)
+EXPECTED (video.buffered.length == '1') OK
+RUN(video.currentTime = video.duration - 0.5)
+EVENT(playing)
+EXPECTED (video.buffered.length == '1') OK
+RUN(video.currentTime = video.duration - 0.5)
.
.
.
(continues until timing out)

[1] - http://trac.webkit.org/changeset/119268/

[Ami Fischman]

Created attachment 145384 [details]
Patch

[Ami Fischman]

Oops, sorry for the noise.
zandobersek: can you verify the patch fixes things for you?

[Ami Fischman]

zandobersek@: BTW, any idea why the GTK EWS didn't catch the problem on 87568?

eric.carlson: i can haz r+?

[Philippe Normand]

GTK EWS doesn't run the layout tests yet. Only Chromium's does, AFAIK.

[Chris Dumez]

Test has been skipped for EFL port in Bug 88194, while waiting for the fix to land.

[WebKit Review Bot]

Comment on attachment 145384 [details]
Patch

Clearing flags on attachment: 145384

Committed r119415: <http://trac.webkit.org/changeset/119415>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.