<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GStreamer] video sink management improvements"
href="https://bugs.webkit.org/show_bug.cgi?id=164709">164709</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[GStreamer] video sink management improvements
</td>
</tr>
<tr>
<th>Classification</th>
<td>Unclassified
</td>
</tr>
<tr>
<th>Product</th>
<td>WebKit
</td>
</tr>
<tr>
<th>Version</th>
<td>WebKit Nightly Build
</td>
</tr>
<tr>
<th>Hardware</th>
<td>Unspecified
</td>
</tr>
<tr>
<th>OS</th>
<td>Unspecified
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>Normal
</td>
</tr>
<tr>
<th>Priority</th>
<td>P2
</td>
</tr>
<tr>
<th>Component</th>
<td>Platform
</td>
</tr>
<tr>
<th>Assignee</th>
<td>webkit-unassigned@lists.webkit.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>pnormand@igalia.com
</td>
</tr>
<tr>
<th>CC</th>
<td>alex@igalia.com, calvaris@igalia.com, cgarcia@igalia.com, pnormand@igalia.com
</td>
</tr></table>
<p>
<div>
<pre>+++ This bug was initially created as a clone of <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GStreamer][OWR] poor video rendering in apprtc"
href="show_bug.cgi?id=164585">Bug #164585</a> +++
Quoting Xabier:
"> Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerOwr.cpp:343
<span class="quote">> + GstElement* sink;</span >
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
"</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>