[Bug 657080] New: aacparse: failing test due to two buffers being dropped for one sync loss

[GStreamer (bugzilla.gnome.org)]

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

           Summary: aacparse: failing test due to two buffers being
                    dropped for one sync loss
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: vincent.penquerch at collabora.co.uk
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


The aacparse test tries to feed a garbage frame in the middle of valid frames,
and expects to receive an output buffer for each valid input buffer.

The aacparse element wants two frames in a row to acquire sync, and ends up
rejecting a valid frame, causing the aacparse test to come up one buffer short.

Disabling the "need second buffer" test in aacparse causes the test to be
happy.

The straightforward fix would be to change the test to know it should expect
one less buffer, but I'm wondering whether it's best to keep track in the
aacparse element whehter we found sync once before, and, if so, resync on a
single frame.
This would mean a bit error would drop only the corrupt frame, and not a valid
one as well.

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