[GStreamer] Use of playbin3 when USE_PLAYBIN3 environment variable is set
Created attachment 337699 [details] Patch
Comment on attachment 337699 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=337699&action=review > Source/WebCore/platform/graphics/gstreamer/GRefPtrGStreamer.h:24 > +#include <gst/gst.h> This is for GST_CHECK_VERSION(), right? Then please remove the now useless forward typedefs :) > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:2402 > +void MediaPlayerPrivateGStreamer::createGSTPlayBin(const gchar *playbin_name) Please use camelCase :P
Comment on attachment 337699 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=337699&action=review >> Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:2402 >> +void MediaPlayerPrivateGStreamer::createGSTPlayBin(const gchar *playbin_name) > > Please use camelCase :P Ah also, we use char instead of gchar usually, and the star is mispositioned. Not sure why check-webkit-style didn't complain there...
Created attachment 337714 [details] Patch
Created attachment 337715 [details] Patch
> Ah also, we use char instead of gchar usually, and the star is mispositioned. Not sure why check-webkit-style didn't complain there... From my small checks, we do using g<ctypes> quite a lot all around :-) Looks like the star positioning is not enforce anywhere in the webkit style (clang-format doesn't seem to care, and the script doesn't either)
(I fixed it all anyway)
Comment on attachment 337715 [details] Patch Clearing flags on attachment: 337715 Committed r230540: <https://trac.webkit.org/changeset/230540>
All reviewed patches have been landed. Closing bug.
<rdar://problem/39354946>
Please, environment variables need to be documented at https://trac.webkit.org/wiki/EnvironmentVariables