[Bug 705991] Adding support for DASH common encryption to qtdemux and dashdemux

[GStreamer (GNOME Bugzilla)]

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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #300676|none                        |committed
             status|                            |

--- Comment #104 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 300676
  --> https://bugzilla.gnome.org/attachment.cgi?id=300676
Adds a new Protection event to GStreamer core

The event stuff looks mostly fine. I have a question though: the event is of
type MULTI_STICKY, presumably so we can have multiple types of protection data
sticky per pad/stream, e.g. some global data plus something that gets updated
on a regular basis. What parallelism will we require in practice though? Only
different origins, or do we expect there may be a need for events for different
system_ids to co-exist for the same stream?

I think it would be nicer if the system_id string and the origin string were
added to the event GstStructure explicitly as fields as well. That would also
simplify the parsing code in parse_protection() and would allow you to return a
pointer to the string without making a copy (which is nicer for the caller).

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