qtdemux: improve fragment-start tracking
authorThiago Santos <ts.santos@sisa.samsung.com>
Tue, 3 Dec 2013 18:08:25 +0000 (15:08 -0300)
committerThiago Santos <ts.santos@sisa.samsung.com>
Wed, 4 Dec 2013 13:36:38 +0000 (10:36 -0300)
commit1fd094d96ba9b77921b087600912475edb31b985
tree3b118bd9f69985ce39d63b7c45e896b4721d3568
parent3c70741e45da94e76b7015c367857312861886ba
qtdemux: improve fragment-start tracking

Some buffers can have multiple moov atoms inside and the strategy
of using the gst_adapter_prev_pts timestamp to get the base timestamp
for the media of the fragment would fail as it would reuse the same
base timestamp for all moofs in the buffer instead of accumulating
the durations for all of them.

Heres a better explanation of the issue:
qtdemux receives a buffer where PTS(buf) = X
buf -> moofA | moofB | moofC

The problem was that PTS(buf) was used as the base timestamp for
all 3 moofs, causing all buffers to be X based. In this case we want
only moofA to be X based as it is what the PTS on buf means, and the
other moofB and moofC just use the accumulated timestamp from the
previous moofs durations.

To solve this, this patch uses gst_adapter_prev_pts distance
result, this allows qtdemux to calculate if it should use the
resulting pts or just accumulate the samples as it can identify
if the moofs belong to the same upstream buffer or not.

https://bugzilla.gnome.org/show_bug.cgi?id=719783
gst/isomp4/qtdemux.c
gst/isomp4/qtdemux.h