[Bug 706441] videomixer: Rework/review the way we handle flushing

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=706441
  GStreamer | gst-plugins-good | 1.x

Alessandro Decina <alessandro.d> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |alessandro.d at gmail.com

--- Comment #5 from Alessandro Decina <alessandro.d at gmail.com> 2013-08-27 20:09:24 UTC ---
(In reply to comment #2)
> I think this should work the following way:
>
> 
> 3) flushing seek: Forward to all upstreams, don't send flush-start/stop
> ourselves downstream. Forward the *first* flush-start and flush-stop that
> happens because of the seek, drop all others. Reset the GstVideoMixer2Pad
> segment when receiving the flush-stops (all of them for the corresponding pad).

Isn't this racy if you have two upstreams where the 1st flushes, sending flush
start/stop and the 2nd still sends some data belonging to the old segment after
the 1st has sent flush_stop (and flush_stop has been propagated downstream)?

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