Bug 205916 – [GStreamer] OGV/VP8 video not playing on minibrowser (neither epiphany)

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Manuel Rego Casasnovas]

The following video doesn't work on minibrowser:
* https://w3c-test.org/media/test.ogv

While the following ones work without problems:
* https://w3c-test.org/media/test.mp4
* https://w3c-test.org/media/video.ogv
* https://w3c-test.org/media/counting.ogv

[Philippe Normand]

Forwarded upstream: https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/719

[Philippe Normand]

(In reply to Manuel Rego Casasnovas from comment #0)
> The following video doesn't work on minibrowser:
> * https://w3c-test.org/media/test.ogv
> 

Cathie, Rego, would you know which encoder produced this file?

[cathiechen]

(In reply to Philippe Normand from comment #2)
> (In reply to Manuel Rego Casasnovas from comment #0)
> > The following video doesn't work on minibrowser:
> > * https://w3c-test.org/media/test.ogv
> > 
> 
> Cathie, Rego, would you know which encoder produced this file?

Hi Philippe,

I checked the file properties which provides some infos. Not sure if this useful or not.

test.ogv:
The video CodeC is VP8 and the audio CodeC is Vorbis.

video.ogv and counting.ogv:
The video CodeC is Theora and the audio CodeC is N/A.

[Philippe Normand]

Doesn't help much unfortunately ;) It would be great to know which software created test.ogv because it's broken.

Nevertheless, I've submitted a workaround to GStreamer so this file plays fine afterwards :)

[Philippe Normand]

If you need test.ogv to be playable for a test or something, please let me know so I can include the GStreamer patch for it in WebKit.

[Manuel Rego Casasnovas]

(In reply to Philippe Normand from comment #5)
> If you need test.ogv to be playable for a test or something, please let me
> know so I can include the GStreamer patch for it in WebKit.

No, not right now as we used a different file for that test in particular (as the specific file didn't matter a lot).

If the file is broken maybe we should just fix it in WPT repository directly. I don't believe it's expected to be a broken file.
The test file comes from https://chromium-review.googlesource.com/c/chromium/src/+/842325/, so if we can generate a "right one" we could send a PR to WPT to modify that file.
Whatever it means that the file is broken or the file is right, but if you have a good explanation for that I guess it'd be fine to change the file in WPT.

[Lauro Moura]

r263987 synced the semantics tests and two tests (1 old, 1 new) use the same media/test.ogv and are timing out with the test promise failing due to "operation not supported error" after trying to play it.

Tests:

imported/w3c/web-platform-tests/html/semantics/embedded-content/media-elements/track/track-element/track-cues-cuechange.html
imported/w3c/web-platform-tests/html/semantics/embedded-content/media-elements/track/track-element/track-cues-cuechange-dynamically-created-track-element.html

I'm marking them as skip for now until either the wpt file is updated or gstreamer patched in the SDK.

[Philippe Normand]

Created attachment 403677 [details]
Patch

[EWS]

Committed r264013: <https://trac.webkit.org/changeset/264013>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 403677 [details].