[Bug 734499] videoaggregator: Does not update segment position from upstream segments

[GStreamer (bugzilla.gnome.org)]

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

Thibault Saunier <tsaunier> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #286777|0                           |1
        is obsolete|                            |

--- Comment #21 from Thibault Saunier <tsaunier at gnome.org> 2014-10-06 07:11:29 UTC ---
Created an attachment (id=287809)
 View: https://bugzilla.gnome.org/attachment.cgi?id=287809
 Review: https://bugzilla.gnome.org/review?bug=734499&attachment=287809

(In reply to comment #20)

> Right, so the difference is that the segment position is not set to -1 on
> FLUSH_STOP like in videomixer2.c:1951.  The aggregator equivalent,
> gst_videoaggregator_flush is never called, presumambly because
> priv->flush_seeking is FALSE due to there being no seek on the pad.

So in videomixer we concider that we are flushed, whenever a sinkpad is
flushed, that sounds conceptually weird to me and I wonder if it is what we
actually want. I attach a patch here that you can check to see if it actually
fixes your issue.

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