Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 0717ba7dec18b44a84043c0b7a83db6f7ae97a62
https://github.com/WebKit/WebKit/commit/0717ba7dec18b44a84043c0b7a83db6f7ae97a62
Author: Philippe Normand <ph...@igalia.com>
Date: 2023-09-04 (Mon, 04 Sep 2023)
Changed paths:
M Source/WebCore/platform/graphics/gstreamer/VideoEncoderGStreamer.cpp
M Source/WebCore/platform/graphics/gstreamer/VideoEncoderGStreamer.h
Log Message:
-----------
[GStreamer][WebCodecs] Build video encoder active configuration from source
pad caps
https://bugs.webkit.org/show_bug.cgi?id=261084
Reviewed by Xabier Rodriguez-Calvar.
The active configuration should now have a valid header buffer and colorspace,
filled from the caps
of the underlying GStreamer encoder source pad. Some encoders store the header
in
`streamheaders` (vp8, vp9) and some use a `codec_data` field (x264), so both
cases need to be
handled.
Drive-by: Weaken the link between the GStreamerInternalVideoEncoder and its
GStreamerElementHarness
in order to avoid memory leaks.
* Source/WebCore/platform/graphics/gstreamer/VideoEncoderGStreamer.cpp:
(WebCore::GStreamerInternalVideoEncoder::create):
(WebCore::GStreamerVideoEncoder::create):
(WebCore::GStreamerVideoEncoder::GStreamerVideoEncoder):
(WebCore::GStreamerInternalVideoEncoder::GStreamerInternalVideoEncoder):
(WebCore::GStreamerInternalVideoEncoder::encode):
* Source/WebCore/platform/graphics/gstreamer/VideoEncoderGStreamer.h:
Canonical link: https://commits.webkit.org/267606@main
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
