[Bug 732265] vaapidecode: h264: add support for decoding MVC base views only

[GStreamer (GNOME Bugzilla)]

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

Víctor Manuel Jáquez Leal <vjaquez at igalia.com> changed:

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

--- Comment #19 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 356381:
 --> (https://bugzilla.gnome.org/review?bug=732265&attachment=356381)

::: gst/vaapi/gstvaapidecode.c
@@ +1169,3 @@
+  caps_new = gst_caps_from_string ("video/x-h264");
+  structure = gst_caps_get_structure (caps_new, 0);
+  gst_structure_set (structure, "profile", G_TYPE_STRING, profile_name, NULL);

Use this 

caps_new = gst_caps_new_simple ("video/x-h264", "profile", profile_name, NULL);

@@ +1222,3 @@
+  }
+
+  if (base_only && !have_mvc && have_high) {

If va backend doesn't profile mvc profiles but has high profile, I would set
base_only as TRUE mandatory.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.