[Bug 723646] New: mpeg4videoparse: push caps before segment when first data is a GAP

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=723646
  GStreamer | gst-plugins-bad | unspecified

           Summary: mpeg4videoparse: push caps before segment when first
                    data is a GAP
    Classification: Platform
           Product: GStreamer
           Version: unspecified
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: thiago.sousa.santos at collabora.co.uk
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


While the attached patch fixes it, I'd rather fix this in the baseclass
if possible because it would not require fixing it in all parsers.
Unfortunately I found no sane way of doing it with the current baseclass
methods. And if we have to add a new method, better just use the
event handler just like this patch? Any ideas other than doing
a check for a src pad caps before pushing a segment?

Another option is to store the GAP as a pending event while caps
aren't decided, but then a GAP can be very long and disrupt playback/preroll
if not pushed. So not a good one.

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