[Bug 780410] qtdemux: distinguish TFDT with value 0 from no TFDT at all

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=780410

--- Comment #1 from Enrique Ocaña González <eocanha at igalia.com> ---
I've thought about two possible solutions:

A) Have an extra boolean parameter to qtdemux_parse_trun() hinting if the
tfdt_node has been found in qtdemux_parse_moof(). Use that info in the
condition added by the mentioned commit, instead of checking decode_ts != 0.
I've coded this solution locally and can confirm that it solves the issue.

B) Initialize decode_time to GST_CLOCK_TIME_NONE in qtdemux_parse_moof() and
add a couple of extra cases in qtdemux_parse_trun() to use 0 when
GST_CLOCK_TIME_NONE is passed. Except in the decode_ts != 0 condition, of
course, which now would be decode_ts != GST_CLOCK_TIME_NONE.

Which one do you prefer me to submit as a patch?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.