Bug 164709 - [GStreamer] video sink management improvements
Summary: [GStreamer] video sink management improvements
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-14 01:52 PST by Philippe Normand
Modified: 2018-02-09 10:57 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2016-11-14 01:52:41 PST
+++ This bug was initially created as a clone of Bug #164585 +++

Quoting Xabier:

"> Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerOwr.cpp:343
> +    GstElement* sink;

This needs to be GRefPtr, why?

createGLAppSink and gst_bin_new return a floating ref.
g_object_get_ returns a hard ref.

I didn't dig into the code of g_object_new/set to know what happens with an object passed as argument to the constructor, if the ref is sinked or not, but g_object_get returns a hard ref so we need to change this.

I am going thru all this and even with this naïve code, the ref counting is incredibly hard to follow :) Not saying that is wrong or that there is another better way to do, just stating the fact, specially because OWR is undocumented and you have to check the code directly
"
Comment 1 Philippe Normand 2018-02-09 10:57:35 PST
OpenWebRTC support was removed. Please reopen if this issue happens elsewhere.