[Bug 642879] qtmux: add a 'variant' with the bare video/quicktime media type

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=642879
  GStreamer | gst-plugins-bad | unspecified

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #181466|none                        |reviewed
             status|                            |

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-03-24 14:02:24 UTC ---
(From update of attachment 181466)
>+        QT_APPLE_VARIANT ("GstQtMuxDefault", "qtmuxdefault", "video/quicktime"),
>+      }
>+  ,
>+  {
>+        QT_APPLE_VARIANT ("GstQtMux", "qtmux",
>+            "video/quicktime, variant=(string)apple"),
>       }
>   ,

So: we need a qtmux* that advertises "video/quicktime" and one that advertises
"video/quicktime,variant=apple", correct? To support re-muxing to the same
format properly (because the typefind caps may or may not have a variant field,
and encodebin needs an exact match).

If I read this patch correctly, it adds a new "qtmuxdefault" element in
addition to the existing "qtmux" element. I'm not sure this is needed. Surely
the "qtmux" element can have "video/quicktime,variant=(string)apple;
video/quicktime" in its template caps, and handle both?

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