[Bug 685812] qtdemux: Forward cropping information downstream

[GStreamer (GNOME Bugzilla)]

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

Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas at ndufresne.ca

--- Comment #4 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
There is no x/y value for this case, so there is no need fro cropping. If
downstream of the decoder support GstVideoMeta, the decoder can just
passthrough that value downstream. If it does not support GstVideoMeta, then
gst_video_frame_copy() will do the job. Currently this works with avdec, didn't
check if it's fully correct or not. It does not work with v4l2videodec yet,
because I only recently learn this case existed.

If there is a x/y value exposed, then it's non-trivial, and crop meta would be
best. But then you are screwed because demuxers cannot query to know if
crop-meta is supported downstream. You have to fix the demuxers, or make this
caps feature. This is a big project. Then, you need to port all the decoder to
support this.

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