qtdemux: set need_segment after a second moov
authorAlicia Boya García <aboya@igalia.com>
Fri, 30 Nov 2018 18:29:30 +0000 (19:29 +0100)
committerSebastian Dröge <slomo@coaxion.net>
Fri, 30 Nov 2018 20:44:57 +0000 (20:44 +0000)
commit38b553dda76651d7c412537c1b095a83761c2d9b
treeacdc8b24cab9175357e967b2c60b1120812bdbfb
parent26cc201c8a42db596c494753d8fc47876874a900
qtdemux: set need_segment after a second moov

stream.segment should be updated with the values of the current edit
list, also when a new `moov` is received. Unfortunately this was not
being the case because of an early return.

As a consequence of this bugs, no end of movie clipping was being
performed on the new moov and no segment event was being emitted.

When performing stream switching (e.g. in MSE) the new moov may have a
different edit list. This is often the case when switching between
baseline H.264 (which lacks B-frames) and more demanding profiles. For
this reason it's important to emit a new segment in order to be able
to get matching stream times.
gst/isomp4/qtdemux.c