[Bug 752413] dashdemux: add support for parsing UTCTiming elements

[GStreamer (GNOME Bugzilla)]

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

--- Comment #12 from A Ashley <bugzilla at ashley-family.net> ---
tldr: yes, clock_drift->method is not used and can be removed.

Essentially all the methods fall into one of four categories:
1. Use an NTP/SNTP server
2. Do an HTTP HEAD request and look at the Date: field
3. Do an HTTP GET request and parse the payload
4. Parse the value directly defined in the element

The patch implements 2 of the 3 payload formats defined for category 3. The
other payload format (ntp format) is slightly ambiguous and I've never seen it
used.

I have only seen categories #3 and #4 used in streams. The reason for choosing
#3 is because it provides a reasonable trade-off between client and server
complexity and seems to be the most widely implemented. #4 provides a lousy
time reference (it can be out by up to mininimumUpdatePeriod seconds) but could
be implemented if this was considered useful.

I looked at implementing #2 but it needed some surgery in gsturidownloader to
pass back the HTTP response headers. Not seen anyone use it, so that was
another reason for not bothering to implement it.

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