[Bug 709241] applemedia: Enable I420 on input pad

[GStreamer (bugzilla.gnome.org)]

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

Dominik Röttsches <dominik.rottsches> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #256238|0                           |1
        is obsolete|                            |

--- Comment #4 from Dominik Röttsches <dominik.rottsches at intel.com> 2013-10-02 11:11:21 UTC ---
Created an attachment (id=256251)
 View: https://bugzilla.gnome.org/attachment.cgi?id=256251
 Review: https://bugzilla.gnome.org/review?bug=709241&attachment=256251

Enabling I420 (3)

(In reply to comment #3)
> Review of attachment 256238 [details]:
> 
> ::: sys/applemedia/vtenc.c
> @@ +800,3 @@
> +        pixel_format_type = kCVPixelFormatType_420YpCbCr8Planar;
> +      } else {
> +        pixel_format_type = kCVPixelFormatType_420YpCbCr8BiPlanarVideoRange;
> 
> For future extensibility please check the video format here :)

Makes sense :-)

> Also is it guaranteed that both are always supported? If not it would make
> sense to implement something to only return what actually is supported from the
> CAPS query.

I have tested this on an iMac and and a Macbook pro - on these Mac OS systems
both always work. I don't have any info on whether these are always supported,
for neither of the formats. I can try to investigate that separately.

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