-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GStreamer] FLAC Audio Format Support #4927
base: main
Are you sure you want to change the base?
Conversation
EWS run on previous version of this PR (hash d611748) |
I'll try to write a test for this... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@philn What browsers do support this? FLAC bytestream is clearly not in the spec registry https://www.w3.org/TR/mse-byte-stream-format-registry/#registry -- and it's not in the MSE WebM ByteStream spec either for that matter https://www.w3.org/TR/mse-byte-stream-format-webm/#webm-mime-parameters (it's not generally a recognized audio format for WebM for that matter) GStreamer gives us a lot of flexibility in adding formats to MSE, both containerized and as raw packetized streams, but I'd like to understand the motivations behind any non-standard additions. |
I found this demo, https://brionv.com/misc/msetest4/flac2.html However this bug came up for a real use-case, it seems Deezer premium streams So yeah... not sure what to do..... |
https://bugs.webkit.org/show_bug.cgi?id=245876 Reviewed by NOBODY (OOPS!). Flac support was added in AppendPipeline, in case the SourceBuffer is audio/flac that would mean typefind should be able to detect it and that the flac data is not muxed, thus requires no demuxing. The MediaSourcePrivateGStreamer::addSourceBuffer() also now returns NotSupported in case the containerType is not supported by AppendPipeline. * Source/WebCore/platform/graphics/gstreamer/mse/AppendPipeline.cpp: (WebCore::AppendPipeline::AppendPipeline): * Source/WebCore/platform/graphics/gstreamer/mse/MediaSourcePrivateGStreamer.cpp: (WebCore::MediaSourcePrivateGStreamer::addSourceBuffer): * Source/WebCore/platform/graphics/gstreamer/mse/SourceBufferPrivateGStreamer.cpp: (WebCore::SourceBufferPrivateGStreamer::isContentTypeSupported): * Source/WebCore/platform/graphics/gstreamer/mse/SourceBufferPrivateGStreamer.h:
EWS run on current version of this PR (hash e0bd875) |
In the bug report, the reporter said this is supported by the other web engines. |
But you also said you couldn't get the test to work in other browsers, so I'm a bit confused. |
e0bd875
e0bd875