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

[GStreamer (GNOME Bugzilla)]

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

--- Comment #8 from A Ashley <bugzilla at ashley-family.net> ---
(In reply to Sebastian Dröge (slomo) from comment #7)
> Review of attachment 309003 [details] [review]:
> 
> ::: ext/dash/gstdashdemux.c
> @@ +1228,3 @@
>    GstDashDemux *dashdemux = GST_DASH_DEMUX_CAST (demux);
> +  return MIN (dashdemux->client->mpd_node->minimumUpdatePeriod * 1000,
> +      CLOCK_UPDATE_INTERVAL);
> 
> Hm but this means that we will also update the manifest each time, right?
> Which might mean that we update the manifest too often, and a server might
> complain about that

Yes it does. However the clock poll is set to 30 minutes, so it's not going to
add much traffic.
> 
> ::: ext/dash/gstmpdparser.c
> @@ +222,3 @@
> +  {"urn:mpeg:dash:utc:http-iso:2014", GST_MPD_UTCTIMING_TYPE_HTTP_ISO},
> +  {"urn:mpeg:dash:utc:http-ntp:2014", GST_MPD_UTCTIMING_TYPE_HTTP_NTP},
> +  {"urn:mpeg:dash:utc:direct:2014", GST_MPD_UTCTIMING_TYPE_DIRECT},
> 
> Who thought that it's a good idea to add that many methods for the same
> thing? :) Not your fault of course but it's a bit... special

The joys of standardisation. Can't agree on a timing method - let's have seven!

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