RESOLVED INVALID 164709
[GStreamer] video sink management improvements 
https://bugs.webkit.org/show_bug.cgi?id=164709
Summary [GStreamer] video sink management improvements
Philippe Normand
Reported 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 "
Attachments
Add attachment proposed patch, testcase, etc.
Philippe Normand
Comment 1 2018-02-09 10:57:35 PST
OpenWebRTC support was removed. Please reopen if this issue happens elsewhere.
Note You need to log in before you can comment on or make changes to this bug.