Bug 235870

Summary: [GStreamer] Support for vpx in webrtc encoder
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: PlatformAssignee: Philippe Normand <pnormand>
Severity: Normal CC: calvaris, cgarcia, eric.carlson, ews-watchlist, glenn, gustavo, hta, jer.noble, menard, philipj, sergio, tommyw, vjaquez, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Philippe Normand 2022-01-30 07:52:35 PST
Comment 1 Philippe Normand 2022-01-30 07:57:55 PST
Created attachment 450360 [details]
Comment 2 Philippe Normand 2022-01-30 08:30:20 PST
Created attachment 450362 [details]
Comment 3 Víctor M. Jáquez L. 2022-02-01 01:53:12 PST
heh... another patch that I cooked long ago and it bitrotten in my branch.
Comment 4 Víctor M. Jáquez L. 2022-02-01 01:56:49 PST
Comment on attachment 450362 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=450362&action=review

> Source/WebCore/platform/mediastream/gstreamer/GStreamerVideoEncoder.cpp:38
> +static GstStaticPadTemplate srcTemplate = GST_STATIC_PAD_TEMPLATE("src", GST_PAD_SRC, GST_PAD_ALWAYS, GST_STATIC_CAPS("video/x-h264;video/x-vp8;video/x-vp9"));

What have worried my about this static (something that had hit me in gstreamer-vaapi)  is that the element is offering all these codecs, but we don't effectively assure we have them before offering them via caps template.
Comment 5 Philippe Normand 2022-02-01 09:13:04 PST
Thanks for raising this issue :) I'll try to address it in a follow-up patch!
Comment 6 EWS 2022-02-01 09:18:21 PST
Committed r288883 (246635@main): <https://commits.webkit.org/246635@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 450362 [details].
Comment 7 Radar WebKit Bug Importer 2022-02-01 09:19:17 PST