[Bug 703499] New: videomixer: take the newly introduced segment->offset into account.

[GStreamer (bugzilla.gnome.org)]

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

           Summary: videomixer: take the newly introduced segment->offset
                    into account.
    Classification: Platform
           Product: GStreamer
           Version: unspecified
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: mathieu.duponchelle at epitech.eu
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


I observed cases where seeking with SEEK_TYPE_NONE in gnonlin would make
the collectpads have a non negative segment offset.

This caused the conversion to running time to return -1, and thus
to make videomixer assert. In my opinion, the check had to be updated because
that offset is a new introduction.

Also, I think the logic is better, we check start against segment->start and
stop against segment->stop, instead of the weird logic that was before and
that actually let one case pass, the one where start_time would be
inferior to segment->start but not end_time.

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