From: Víctor Manuel Jáquez Leal <victorx.jaquez@intel.com>
Date: Wed, 14 Oct 2015 18:22:43 +0000 (+0200)
Subject: vaapidecode: set format before decide allocation
X-Git-Tag: 1.19.3~503^2~1631
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=45487fe87cc4dcc0c319701af0516ef6973794ec;p=platform%2Fupstream%2Fgstreamer.git

vaapidecode: set format before decide allocation

There is a regression from commit 3d8e5e. It was expected the buffer pool
allocation occur before the caps negotiation, but it is not.

This patch fixes this regression: the caps negotiation is done regardless the
allocation query from downstream.

Signed-off-by: Víctor Manuel Jáquez Leal <victorx.jaquez@intel.com>

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

diff --git a/gst/vaapi/gstvaapidecode.c b/gst/vaapi/gstvaapidecode.c
index 2dc7c6c..2830277 100644
--- a/gst/vaapi/gstvaapidecode.c
+++ b/gst/vaapi/gstvaapidecode.c
@@ -213,12 +213,9 @@ gst_vaapidecode_update_src_caps (GstVaapiDecode * decode)
   switch (feature) {
 #if (USE_GLX || USE_EGL)
     case GST_VAAPI_CAPS_FEATURE_GL_TEXTURE_UPLOAD_META:
-      if (decode->has_texture_upload_meta)
         features =
             gst_caps_features_new
             (GST_CAPS_FEATURE_META_GST_VIDEO_GL_TEXTURE_UPLOAD_META, NULL);
-      else
-        format = GST_VIDEO_FORMAT_I420;
       break;
 #endif
 #if GST_CHECK_VERSION(1,5,0)