[Bug 677535] isomp4/qtdemux: Add support for MPEG DASH tfdt box

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=677535
  GStreamer | gst-plugins-good | 0.10.31

--- Comment #9 from David Corvoysier <david.corvoysier at orange.com> 2012-08-14 09:59:05 UTC ---
> Note that position tracking is done by the 'time/position' member in a segment
> (event), so rather than a segment (S, NONE, S) and buffer times starting at S,
> it is also possible (and maybe easier/cleaner) to use (0, NONE, S) and buffer
> times starting at 0.

...

> However, in combination with above remark it should suffice to have atom
> parsing dump the tfdt decode/position in some state variable, which is then
> used as the position parameter for the (initial) newsegment event (created in
> the _STATE_HEADER handling).

Thanks for pointing that out: this is makes indeed more sense adjusting only
playback position (and it is alos simpler). I have still however a few
questions about how it could be implemented.

First, where would it make more sense to store the parsed decode/position ? In
the qtdemux object as a new member ?

Second, the new segment event is currently created when the moov has been
parsed, but the tfdt is retrieved from the moof, leaving two options:
A - Move event creation code so that it executes after the moof has been parsed
B - Leave event creation code as is and Adjust time/position when the moov has
been parsed.
Which option would you suggest ?

I have attached a temporary updated patch where I implement option B, but still
leaving the event adjustment in the low-lvel parsing code (which I agree is bad
design).

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.