We should support the 0.11 APIs. A ./configure --with-gstreamer=0.11 could be used to build (defaulting to 0.10). For video rendering we will need a new video-sink that will initially provide the same features as the current 0.10 video-sink.
Also, the GStreamerGWorld module is not really worth porting as it's planned to be removed soon.
Created attachment 123944 [details] Patch First iteration :)
I can probably find a way to split this patch if needed.
Attachment 123944 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'Source/WebCore/ChangeLog', u..." exit_code: 1 Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:67: webkit_video_sink_signals is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:71: timeout_id is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:72: buffer_mutex is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:73: data_cond is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:331: need_pool is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:362: n_param_values is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:363: invocation_hint is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:387: gobject_class is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:388: gstbase_sink_class is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer1.cpp:389: element_class is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Total errors found: 10 in 22 files If any of these errors are false positives, please file a bug against check-webkit-style.
I'm going to split this in 4 patches: - build infrastructure - webkitwebsrc changes - video-sink - disable GStreamerGWorld
Comment on attachment 123944 [details] Patch Went through the patch split machine.
Last patch landed. Next step would be to have test coverage on the bots. I'll file a separate bug for this.