matroskamux: Always write the default frame duration for VP8/9 too
authorSebastian Dröge <sebastian@centricular.com>
Thu, 29 Sep 2016 07:19:56 +0000 (10:19 +0300)
committerSebastian Dröge <sebastian@centricular.com>
Thu, 29 Sep 2016 07:19:56 +0000 (10:19 +0300)
The WebM spec allows this now, and it allows us to guess a framerate.

See https://bugzilla.gnome.org/show_bug.cgi?id=772141 and
also https://bugzilla.gnome.org/show_bug.cgi?id=654379

gst/matroska/matroska-mux.c

index 4922496..f77bd49 100644 (file)
@@ -357,7 +357,6 @@ gst_matroska_mux_class_init (GstMatroskaMuxClass * klass)
  * Start of pad option handler code
  */
 #define DEFAULT_PAD_FRAME_DURATION TRUE
-#define DEFAULT_PAD_FRAME_DURATION_VP8 FALSE
 
 enum
 {
@@ -978,13 +977,6 @@ gst_matroska_mux_video_pad_setcaps (GstPad * pad, GstCaps * caps)
   videocontext->pixel_width = width;
   videocontext->pixel_height = height;
 
-  /* set vp8 defaults or let user override it */
-  if (GST_MATROSKAMUX_PAD_CAST (pad)->frame_duration_user == FALSE
-      && (!strcmp (mimetype, "video/x-vp8")
-          || !strcmp (mimetype, "video/x-vp9")))
-    GST_MATROSKAMUX_PAD_CAST (pad)->frame_duration =
-        DEFAULT_PAD_FRAME_DURATION_VP8;
-
   if (GST_MATROSKAMUX_PAD_CAST (pad)->frame_duration
       && gst_structure_get_fraction (structure, "framerate", &fps_n, &fps_d)
       && fps_n > 0) {