Bug 68536 – [GTK] media/media-blocked-by-beforeload.html is flaky

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Philippe Normand]

--- /var/lib/buildbot/build/gtk-linux-32-release/build/layout-test-results/media/media-blocked-by-beforeload-expected.txt 
+++ /var/lib/buildbot/build/gtk-linux-32-release/build/layout-test-results/media/media-blocked-by-beforeload-actual.txt 
@@ -30,10 +30,6 @@
 
 EVENT('loadstart')
 
-EVENT('error')
-EXPECTED (event.target.tagName == 'SOURCE') OK
-EXPECTED (video.error == 'null') OK
-
 EVENT('loadedmetadata')
 
 END OF TEST

I've seen other occurences where a load event was fired. This is probably due to an issue in the GStreamer player. Will skip the test for now though.

[Shadi]

Bug 63558 https://bugs.webkit.org/show_bug.cgi?id=63558 has been updated to solve the flakiness of the test.

Philippe, do you want to revert skipping the test?

(In reply to comment #0)
> --- /var/lib/buildbot/build/gtk-linux-32-release/build/layout-test-results/media/media-blocked-by-beforeload-expected.txt 
> +++ /var/lib/buildbot/build/gtk-linux-32-release/build/layout-test-results/media/media-blocked-by-beforeload-actual.txt 
> @@ -30,10 +30,6 @@
> 
>  EVENT('loadstart')
> 
> -EVENT('error')
> -EXPECTED (event.target.tagName == 'SOURCE') OK
> -EXPECTED (video.error == 'null') OK
> -
>  EVENT('loadedmetadata')
> 
>  END OF TEST
> 
> I've seen other occurences where a load event was fired. This is probably due to an issue in the GStreamer player. Will skip the test for now though.

[Martin Robinson]

(In reply to comment #1)
> Bug 63558 https://bugs.webkit.org/show_bug.cgi?id=63558 has been updated to solve the flakiness of the test.
> 
> Philippe, do you want to revert skipping the test?

Sure, go ahead. We can reskip it if there are problems.

[Philippe Normand]

(In reply to comment #2)
> (In reply to comment #1)
> > Bug 63558 https://bugs.webkit.org/show_bug.cgi?id=63558 has been updated to solve the flakiness of the test.
> > 
> > Philippe, do you want to revert skipping the test?
> 
> Sure, go ahead. We can reskip it if there are problems.

Commited http://trac.webkit.org/changeset/104164