Bug 205916 - [GStreamer] OGV/VP8 video not playing on minibrowser (neither epiphany)
Summary: [GStreamer] OGV/VP8 video not playing on minibrowser (neither epiphany)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philippe Normand
URL:
Keywords:
Depends on:
Blocks: 214038
  Show dependency treegraph
 
Reported: 2020-01-08 05:20 PST by Manuel Rego Casasnovas
Modified: 2020-07-07 04:08 PDT (History)
5 users (show)

See Also:


Attachments
Patch (4.99 KB, patch)
2020-07-07 02:04 PDT, Philippe Normand
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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].