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)
commiteb97bb0adbd24f06c6bb32155b27bce995cd9a2d
tree0c69a9d3654ed1cd6c27f33cb1016b06194738cf
parent336ca3207a2ba591107ceba8921ed73957ddd235
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