Bug 68536
| Summary: | [GTK] media/media-blocked-by-beforeload.html is flaky | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Philippe Normand <pnormand> |
| Component: | Media | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | leandro, mrobinson, rakuco, shadi |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 75599 | ||
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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