From dcd277b9160f92e051097210889b969a059a162d Mon Sep 17 00:00:00 2001
From: Wim Taymans <wim.taymans@collabora.co.uk>
Date: Thu, 1 Mar 2012 17:36:08 +0100
Subject: [PATCH] videoconvert: proxy allocation meta when we can

Proxy all the metadata APIs in the allocation query.
Remove all metadata that is dependent on the colorspace, copy others.
---
 gst/videoconvert/gstvideoconvert.c | 84 ++++++++++++++++++++++++++++++++++++--
 1 file changed, 80 insertions(+), 4 deletions(-)

diff --git a/gst/videoconvert/gstvideoconvert.c b/gst/videoconvert/gstvideoconvert.c
index d337797..b492392 100644
--- a/gst/videoconvert/gstvideoconvert.c
+++ b/gst/videoconvert/gstvideoconvert.c
@@ -49,6 +49,12 @@ GST_DEBUG_CATEGORY (videoconvert_debug);
 #define GST_CAT_DEFAULT videoconvert_debug
 GST_DEBUG_CATEGORY_EXTERN (GST_CAT_PERFORMANCE);
 
+GType gst_video_convert_get_type (void);
+
+static GQuark _colorspace_quark;
+
+#define gst_video_convert_parent_class parent_class
+G_DEFINE_TYPE (GstVideoConvert, gst_video_convert, GST_TYPE_VIDEO_FILTER);
 
 enum
 {
@@ -72,8 +78,6 @@ GST_STATIC_PAD_TEMPLATE ("sink",
     GST_STATIC_CAPS (CSP_VIDEO_CAPS)
     );
 
-GType gst_video_convert_get_type (void);
-
 static void gst_video_convert_set_property (GObject * object,
     guint property_id, const GValue * value, GParamSpec * pspec);
 static void gst_video_convert_get_property (GObject * object,
@@ -305,8 +309,74 @@ invalid_palette:
   }
 }
 
-#define gst_video_convert_parent_class parent_class
-G_DEFINE_TYPE (GstVideoConvert, gst_video_convert, GST_TYPE_VIDEO_FILTER);
+static gboolean
+gst_video_convert_propose_allocation (GstBaseTransform * trans,
+    GstQuery * decide_query, GstQuery * query)
+{
+  gboolean ret;
+  guint i, n_metas;
+
+  /* let parent handle */
+  ret = GST_BASE_TRANSFORM_CLASS (parent_class)->propose_allocation (trans,
+      decide_query, query);
+  /* error or passthrough, we're done */
+  if (!ret || decide_query == NULL)
+    return ret;
+
+  /* non-passthrough, copy all metadata, decide_query does not contain the
+   * metadata anymore that depends on the buffer memory */
+  n_metas = gst_query_get_n_allocation_metas (decide_query);
+  for (i = 0; i < n_metas; i++) {
+    GType api;
+
+    api = gst_query_parse_nth_allocation_meta (decide_query, i);
+    gst_query_add_allocation_meta (query, api);
+  }
+  return ret;
+}
+
+typedef struct
+{
+  GstBaseTransform *trans;
+  GstBuffer *outbuf;
+  GQuark tag;
+} CopyMetaData;
+
+static gboolean
+foreach_metadata (GstBuffer * inbuf, GstMeta ** meta, gpointer user_data)
+{
+  CopyMetaData *data = user_data;
+  const GstMetaInfo *info = (*meta)->info;
+
+  if (info->transform_func) {
+    if (gst_meta_api_type_has_tag (info->api, data->tag)) {
+      /* metadata depends on colorspace. FIXME discard for now until we
+       * have some transform data for it. */
+    } else {
+      GstMetaTransformCopy copy_data = { 0, -1 };
+      /* simply copy then */
+      info->transform_func (data->outbuf, *meta, inbuf,
+          _gst_meta_transform_copy, &copy_data);
+    }
+  }
+  return TRUE;
+}
+
+static gboolean
+gst_video_convert_copy_metadata (GstBaseTransform * trans,
+    GstBuffer * inbuf, GstBuffer * outbuf)
+{
+  CopyMetaData data;
+
+  data.trans = trans;
+  data.outbuf = outbuf;
+  data.tag = _colorspace_quark;
+
+  gst_buffer_foreach_meta (inbuf, foreach_metadata, &data);
+
+  return GST_BASE_TRANSFORM_CLASS (parent_class)->copy_metadata (trans, inbuf,
+      outbuf);
+}
 
 static void
 gst_video_convert_finalize (GObject * obj)
@@ -347,6 +417,10 @@ gst_video_convert_class_init (GstVideoConvertClass * klass)
       GST_DEBUG_FUNCPTR (gst_video_convert_transform_caps);
   gstbasetransform_class->fixate_caps =
       GST_DEBUG_FUNCPTR (gst_video_convert_fixate_caps);
+  gstbasetransform_class->propose_allocation =
+      GST_DEBUG_FUNCPTR (gst_video_convert_propose_allocation);
+  gstbasetransform_class->copy_metadata =
+      GST_DEBUG_FUNCPTR (gst_video_convert_copy_metadata);
 
   gstbasetransform_class->passthrough_on_same_caps = TRUE;
 
@@ -428,6 +502,8 @@ plugin_init (GstPlugin * plugin)
   GST_DEBUG_CATEGORY_INIT (videoconvert_debug, "videoconvert", 0,
       "Colorspace Converter");
 
+  _colorspace_quark = g_quark_from_static_string ("colorspace");
+
   return gst_element_register (plugin, "videoconvert",
       GST_RANK_NONE, GST_TYPE_VIDEO_CONVERT);
 }
-- 
2.7.4