Bug 66936 – [GStreamer] fullscreen video pause/play fails

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Philippe Normand]

In fullscreen the webkitsink goes to PAUSE asynchronously whereas autovideosink goes to PAUSE just fine. Then if we want to resume playback webkitsink tries to go to PLAYING again and is starved because buffers are blocked by the identity element upstream :(

I think we can just avoid the use of identity and in the webkitsink skip rendering of the buffers if we're fullscreen already.

[Philippe Normand]

Created attachment 105177 [details]
proposed patch

[Philippe Normand]

Comment on attachment 105177 [details]
proposed patch

Clearing r?, I'll provide a LayoutTest for this patch

[Philippe Normand]

Hum I failed to come up with a test showing the issue, in fact it's pretty internal to the MediaPlayerPrivate, the video appears freezed but the MediaPlayer continues to issue timeupdate events.

[Philippe Normand]

Created attachment 105341 [details]
proposed patch

[Philippe Normand]

Created attachment 105342 [details]
proposed patch

[Alexis Menard (darktears)]

Comment on attachment 105342 [details]
proposed patch

How this relate to https://bugs.webkit.org/show_bug.cgi?id=58548 ?

I can give a try.

Otherwise the change looks good to me.

[Philippe Normand]

(In reply to comment #6)
> (From update of attachment 105342 [details])
> How this relate to https://bugs.webkit.org/show_bug.cgi?id=58548 ?
> 

That's a different issue, bug 58548 is related to the segment handling and seeking. I need to get back to that one :)

> I can give a try.
> 
> Otherwise the change looks good to me.

Yes, please try on Qt!

[Martin Robinson]

Comment on attachment 105342 [details]
proposed patch

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

This looks sane to me.

> Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer.cpp:377
>  GstElement*
> -webkit_video_sink_new(void)
> +webkit_video_sink_new(WebCore::GStreamerGWorld* gstGWorld)

This should be one line.

> Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer.cpp:379
> +    GstElement* element = (GstElement*)g_object_new(WEBKIT_TYPE_VIDEO_SINK, 0);

Please use a C++-style cast or if GST_ELMENT is available, use that.

> Source/WebCore/platform/graphics/gstreamer/VideoSinkGStreamer.cpp:381
> +    WebKitVideoSink* sink = WEBKIT_VIDEO_SINK(element);
> +    sink->priv->gstGWorld = gstGWorld;

This should be one line.

[Philippe Normand]

Committed r96310: <http://trac.webkit.org/changeset/96310>