videoaggregator: Fix mixup of running times and segment positions
authorSebastian Dröge <sebastian@centricular.com>
Fri, 11 Sep 2015 10:22:51 +0000 (12:22 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Mon, 14 Sep 2015 17:56:43 +0000 (19:56 +0200)
commit895e71e810650e5d353ed60cf67e69fd0d02e2cf
tree0f740d199886b127bb75b05da87dc43d176170b1
parentac503aee7d958672aa909e0554b4aee5258cd142
videoaggregator: Fix mixup of running times and segment positions

We have to queue buffers based on their running time, not based on
the segment position.

Also return running time from GstAggregator::get_next_time() instead of
a segment position, as required by the API.

Also only update the segment position after we pushed a buffer, otherwise
we're going to push down a segment event with the next position already.

https://bugzilla.gnome.org/show_bug.cgi?id=753196
gst-libs/gst/video/gstvideoaggregator.c