Bug 205916

Summary:

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

Product:

WebKit

Reporter:

Manuel Rego Casasnovas <rego>

Component:

WebKitGTK

Assignee:

Philippe Normand <pnormand>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bugs-noreply, calvaris, cathiechen, lmoura, pnormand

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

214038

Attachments:

Description	Flags
Patch	none

Description Manuel Rego Casasnovas 2020-01-08 05:20:27 PST

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

Comment 1 Philippe Normand 2020-01-08 06:12:28 PST

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

Comment 2 Philippe Normand 2020-01-13 01:04:54 PST

(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?

Comment 3 cathiechen 2020-01-13 01:45:08 PST

(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.

Comment 4 Philippe Normand 2020-01-13 01:52:33 PST

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 :)

Comment 5 Philippe Normand 2020-01-13 06:38:51 PST

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.

Comment 6 Manuel Rego Casasnovas 2020-01-14 01:35:13 PST

(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.

Comment 7 Lauro Moura 2020-07-06 20:23:22 PDT

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.

Comment 8 Philippe Normand 2020-07-07 02:04:34 PDT

Created attachment 403677 [details]
Patch

Comment 9 EWS 2020-07-07 04:08:25 PDT

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

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