[Bug 749581] rtpbasepayload: Try harder to reuse previously configured caps values and give more preference to anything set as properties

[GStreamer (GNOME Bugzilla)]

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

--- Comment #1 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Created attachment 303594
  --> https://bugzilla.gnome.org/attachment.cgi?id=303594&action=edit
rtpbasepayload: Try harder to reuse previously configured caps values and give
more preference to anything set as properties

This affects the pt, ssrc, seqnum-offset and timestamp-offset properties. If
they were set from a property, or we configured caps before, we try to use
that value for them. Even if the first structure of the downstream caps
specifies a different value, we check if the value is supported by other
structures.
Only if all this fails, we use the values given by downstream in the first
structure, i.e. if no properties were set and these are the first caps we
negotiate or downstream does not support our values.

By doing this we ensure that we don't spuriously change ssrcs or other fields
in the middle of the stream (and also consider property values more). Ssrc
changes would currently happen after sending an RTX packet (thus creating a
new internal source inside the rtpsession), and then renegotiating the
payloader (which then gets the RTX ssrc from rtpsession).

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