[MSE][GStreamer] Don't use vorbisparse

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Alicia Boya García]

This patch has been splitted from the original WebKitMediaSrc rework
patch (https://bugs.webkit.org/show_bug.cgi?id=199719).

Unlike other parsers, vorbisparse has latency (in the sense that when
it gets a chain call with a series of complete frames, it may not emit
the parsed frames until another chain in the future), which makes it
inappropriate for AppendPipeline, as there is no good way I know to
flush it.

But actually vorbisparse is not known to be necessary and it was only
introduced for consistency with other formats. Parsers are used in
AppendPipeline to reconstruct information that is lost due to poor
muxes. There have been no reported cases of this being a problem with
Vorbis in WebM, so I'm just removing the parser.

Fixes imported/w3c/web-platform-tests/media-source/mediasource-config-change-webm-a-bitrate.html

[Alicia Boya García]

Created attachment 376042 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 376042 [details]
Patch

Clearing flags on attachment: 376042

Committed r248521: <https://trac.webkit.org/changeset/248521>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.