[Bug 662330] New: Auto-plugger should link first then test

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=662330
  GStreamer | gst-plugins-base | 0.10.x

           Summary: Auto-plugger should link first then test
    Classification: Platform
           Product: GStreamer
           Version: 0.10.x
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: nicolas.dufresne at collabora.co.uk
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=199584)
 View: https://bugzilla.gnome.org/attachment.cgi?id=199584
 Review: https://bugzilla.gnome.org/review?bug=662330&attachment=199584

playbin2: link elements before testing ready state

One of the basic rule in GStreamer is to allocate the resources (e.g. open the
X11 display) in the transition from NULL to READY. The auto-plugger take
advantage of this by doing the state transition before adding and linking the
element. The problem with that approach arrive when an element must share a
context (see GstVideoContext proposal, bug #662321). As the element is not
linked, it's not possible to query neighbour elements or application for an
already allocated video context. This imply implementing hacks in the element
to only allocate the display upon linking (e.g during a call to get_caps() if
in ready state). As this is not straightforward and contributes to make things
unclear, I propose to filter errors on elements and link the element before
testing the ready state.

Note that more patches are required in GstPlaySink, audovideosrc and other
auto-plugger to fully move to this approach.

-- 
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.