[gstreamer-bugs] [Bug 624266] [playbin2] Internal uridecodebin are not properly removed in READY=>NULL

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=624266
  GStreamer | gst-plugins-base | git

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |slomo at circular-chaos.org
     Ever Confirmed|0                           |1

--- Comment #2 from Sebastian Dröge <slomo at circular-chaos.org> 2010-07-13 18:49:57 UTC ---
This patch is not entirely correct. The uridecodebins will usually be removed
from the bin already, which gives you an assertion failure.

e.g. in activate_group() if the suburidecodebin fails and in deactivate_group()
(which is called on state change failures and when switching groups).

At least you should check if the parent of the uridecodebins still is playbin2
before removing them.


Now the real question is, why the old, failed uridecodebins are not removed
from the bin for you.

Of course the idea of the patch is correct and the uridecodebins should be
removed from playbin2 in the state change function when going to NULL but
something is really wrong if the broken uridecodebins are still inside the bin
at that point.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.