[Bug 741030] theoradec: Sets video-meta width/height from padded values

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=741030
  GStreamer | gst-plugins-good | git master

--- Comment #18 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2015-02-03 08:25:54 UTC ---
As I mentioned on IRC before, I think this should work the following way:
- CAPS event contains the display width/height (otherwise applications will
become unhappy, GstDiscoverer reports wrong information, etc)
- caps inside the ALLOCATION query contain the memory width/height
- GstVideoMeta contains the memory width/height
- GstVideoCropMeta contains how much should be cropped relative to the values
inside the video meta and the ALLOCATION query caps

Every other way of doing things will break in one way or another and is
inconsistent.


The above requires that every element that claims to support the CROP meta to
behave like this instead of doing random things like they do currently. Every
element that actually handles the CROP meta will need all 3 information
available anyway (display size, memory size, crop size... and from 2 of this
you can calculate the other).



As a side note, this requires to pass the GstVideoInfo from the memory
width/height caps to be passed to gst_video_frame_map()... which is the only
reasonable value anyway. The display caps have completely different strides,
plane offsets, etc.

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