Bug 154390
Summary: | [GStreamer][HLS] 1.6.3 regressions | ||
---|---|---|---|
Product: | WebKit | Reporter: | Philippe Normand <pnormand> |
Component: | Platform | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WONTFIX | ||
Severity: | Normal | CC: | bugs-noreply, clopez, csaavedra, cturner, eocanha, mcatanzaro, philn |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 174458 | ||
Bug Blocks: |
Philippe Normand
After the build bots were updated from GStreamer 1.4 to 1.6.3 the following http/tests/media tests are failing:
[1/22] http/tests/media/reload-after-dialog.html failed unexpectedly (test timed out, text diff)
[2/22] http/tests/media/video-referer.html failed unexpectedly (text diff)
[3/22] http/tests/media/video-cancel-load.html failed unexpectedly (test timed out, text diff)
[4/22] http/tests/media/video-play-stall-before-meta-data.html failed unexpectedly (text diff)
[5/22] http/tests/media/video-useragent.html failed unexpectedly (text diff)
[6/22] http/tests/media/video-buffered-range-contains-currentTime.html failed unexpectedly (test timed out, text diff)
[7/22] http/tests/media/video-load-suspend.html failed unexpectedly (test timed out, text diff)
[8/22] http/tests/media/video-query-url.html failed unexpectedly (text diff)
[9/22] http/tests/media/video-error-abort.html failed unexpectedly (test timed out, text diff)
[10/22] http/tests/media/remove-while-loading.html failed unexpectedly (test timed out, text diff)
[11/22] http/tests/media/video-throttled-load-metadata.html failed unexpectedly (text diff)
[12/22] http/tests/media/video-served-as-text.html failed unexpectedly (text diff)
[13/22] http/tests/media/video-load-twice.html failed unexpectedly (test timed out, text diff)
[14/22] http/tests/media/video-auth.html failed unexpectedly (text diff)
[15/22] http/tests/media/video-redirect.html failed unexpectedly (text diff)
[16/22] http/tests/media/hls/video-controller-getStartDate.html failed unexpectedly (test timed out, text diff)
[17/22] http/tests/media/hls/hls-audio-tracks-locale-selection.html failed unexpectedly (test timed out, text diff)
[18/22] http/tests/media/hls/hls-video-resize.html failed unexpectedly (test timed out, text diff)
[19/22] http/tests/media/hls/hls-audio-tracks.html failed unexpectedly (test timed out, text diff)
[20/22] http/tests/media/hls/hls-audio-tracks-has-audio.html failed unexpectedly (test timed out, text diff)
[21/22] http/tests/media/hls/hls-accessiblity-describes-video.html failed unexpectedly (test timed out, text diff)
[22/22] http/tests/media/hls/video-cookie.html failed unexpectedly (text diff)
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Carlos Alberto Lopez Perez
http/tests/security/local-video-source-from-remote.html is timing out since then.
Michael Catanzaro
The following tests have never once failed on the bots since they were marked as failing in r196804 (update to GStreamer 1.6.3):
http/tests/media/reload-after-dialog.html
http/tests/media/remove-while-loading.html
http/tests/media/video-auth.html
http/tests/media/video-play-stall-before-meta-data.html
The following tests initially consistently passed, and failed only during a small range of revisions (timeouts during r197752-r197860). They are now consistently passing:
http/tests/media/video-error-abort.html
http/tests/media/video-load-twice.html
The following tests almost always pass, but they are flaky:
http/tests/media/video-buffered-range-contains-currentTime.html
http/tests/media/video-cancel-load.html
Updated expectations in r199270.
Michael Catanzaro
(In reply to comment #2)
> The following tests initially consistently passed, and failed only during a
> small range of revisions (timeouts during r197752-r197860). They are now
> consistently passing:
Also http/tests/media/video-load-suspend.html. Updated expectation in r199273.
Claudio Saavedra
We're now depending on 1.10.4 so this needs review. I unmarked a bunch that were passing for a while now:
http://trac.webkit.org/changeset/218493
Charlie Turner
I moved http/tests/media/hls/hls-video-resize.html into https://bugs.webkit.org/show_bug.cgi?id=199617 since it doesn't appear to be a valid test.
Enrique Ocaña
Now only these tests are marked as skipped by this bug in LayoutTests/platform/glib/TestExpectations:
http/tests/media/hls/video-controller-getStartDate.html
http/tests/media/hls/hls-audio-tracks-locale-selection.html
Enrique Ocaña
Regarding video-controller-getStartDate.html, I've found that getStartDate() returns "Invalid Date" because it's not implemented in MediaPlayerPrivateGStreamer, and can't ever be while the patch from https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/issues/195 isn't merged in GStreamer.
Enrique Ocaña
Submitted https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1417
Enrique Ocaña
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1417 got finally merged today as https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/a2990020b2fee8312f3eb6c8b1a767792403f327 and https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/aafe07a802f7d6790bf73145244835f254a1e40b, and will be available on the next GStreamer 1.22 release in the coming days.