libwebrtc/GStreamerVideoEncoder is written in a GStreamer code-style. It would be better to refactor it using C++/WebKit style, as the other GStreamer elements inside WebKit.
Created attachment 413694 [details]
Let's wait for Phil and Thibault's comments, since IIRC, it was their idea to keep it this way to ease a possible upstreaming process to GStreamer eventually.
(In reply to Xabier Rodríguez Calvar from comment #2)
> Let's wait for Phil and Thibault's comments, since IIRC, it was their idea
> to keep it this way to ease a possible upstreaming process to GStreamer
It indeed was the plan and the reason why we used the GStreamer coding style. I never took the time to make it happen and am not 100% sure what other GStreamer maintainer will think of it so I have no hard opinion at that point.
Comment on attachment 413694 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=413694&action=review
> + GST_STATIC_PAD_TEMPLATE ("src", GST_PAD_SRC, GST_PAD_ALWAYS, GST_STATIC_CAPS ("video/x-h264;video/x-vp8;video/x-vp9"));
vp9 is added here, but I see no mention of a vp9enc below. Maybe add vp9 in a follow-up patch, if this patch is meant to be a refactoring only.
Created attachment 414799 [details]
Committed r270187: <https://trac.webkit.org/changeset/270187>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 414799 [details].