Bug 167109
| Summary: | [Gstreamer] [MSE] [GTK] test imported/w3c/web-platform-tests/media-source/mediasource-seek-beyond-duration.html crashes | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> |
| Component: | Platform | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bugs-noreply, cgarcia, eocanha |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=167108 https://bugs.webkit.org/show_bug.cgi?id=165394 |
||
Carlos Alberto Lopez Perez
Regressions: Unexpected crashes (1)
imported/w3c/web-platform-tests/media-source/mediasource-seek-beyond-duration.html [ Crash ]
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Carlos Alberto Lopez Perez
The test timeouts sometimes also
Carlos Alberto Lopez Perez
Updated TestExpectations file in r210794: <http://trac.webkit.org/changeset/210794>
Enrique Ocaña
The reason why all these imported/w3c/web-platform-tests/media-source tests should be skipped is explained in https://bugs.webkit.org/attachment.cgi?id=297003&action=diff (line 234): "Webm is required by the tests but we don't support it."
Please, skip these tests again until webm support is enabled.
Carlos Alberto Lopez Perez
(In reply to comment #3)
> The reason why all these imported/w3c/web-platform-tests/media-source tests
> should be skipped is explained in
> https://bugs.webkit.org/attachment.cgi?id=297003&action=diff (line 234):
> "Webm is required by the tests but we don't support it."
>
> Please, skip these tests again until webm support is enabled.
Of those 54 tests there are 18 passing (even without webm support). How is that?
Also I don't think skipping tests is the best idea. If you skip tests you pretty much don't have any coverage regarding them. You will miss any revision causing them to suddenly crash or timeout. By marking them as failing or crashing you get basic test coverage for them even when they are still not working or the feature is still not 100% implemented.
Carlos Garcia Campos
This no longer crashes, but fails since we upgraded to GST 1.10.4, see bug #171726
Carlos Alberto Lopez Perez
This bug shouldn't be closed.
There are still many tests pointing here in LayoutTests/platform/gtk/TestExpectations
Carlos Alberto Lopez Perez
(In reply to Carlos Alberto Lopez Perez from comment #6)
> This bug shouldn't be closed.
>
> There are still many tests pointing here in
> LayoutTests/platform/gtk/TestExpectations
Sorry.. i cunfused the last digit of the bug