[Bug 650406] New: vorbisdec does not handle headers in caps

[GStreamer (bugzilla.gnome.org)]

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

           Summary: vorbisdec does not handle headers in caps
    Classification: Platform
           Product: GStreamer
           Version: 0.10.34
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: aleix at oblong.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


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

vorbisdec handling of header in caps

vorbisdec seems to always expect header packets before data packets. If a data
packet is received (a no headers are received yet) the following error message
shows up:

gstvorbisdec.c(976): vorbis_handle_data_packet ():
/GstPipeline:pipeline1/GstDecodeBin2:decodebin21/GstVorbisDec:vorbisdec0:
no header sent yet

However vorbisenc does put vorbis headers (identification, comment and
bitstream codebook) inside buffer caps. This means vorbisdec could make use of
them and initialize the decoder before handling the data packet.

The following patch is a first try to solve this. I'm really new to gstreamer
so it might be better ways to do this, or may be this does not solves the issue
completely.

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