[Bug 684237] videomixer: Caps negotiation does not always work

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #14 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2012-09-21 13:52:12 UTC ---
>From the docs/design/part-negotiation.txt, the recommend way (i.e. convention)
is described. You are right that caps with extra field are compatible (let's
get rid of the word subset, as it's doing subset of values, not fields).
Obviously, the query was a downstream caps query. The way the videomixer is
written, introducing new downstream caps, also introduce them upstream.
Introducing them upstream will make the capabilities incompatible. Basically,
what I'm trying to explain, is that I prefer not introducing new fields in the
first place. Trying and workaround it in getcaps seems odd. So far, I haven't
seen any other use of gst_video_info_to_caps(), so I can't argue on the right
way of using it, maybe it should have had some filter parameter or something.
Otherwise, if all those field are consider standard (personnally I have no idea
what they are used for), then they should probably be included by
GST_VIDEO_CAPS_MAKE() macro no ?

-- 
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.