[Bug 749655] vtdec: Require width and height field for H264

[GStreamer (GNOME Bugzilla)]

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

--- Comment #6 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> ---
Ok, I've looked in master, and interestingly Wim reverted that work. As an
example, for the hypothetical pipeline:

  someh264src ! rtph264pay ! rtph264depay ! vtdec ! ...

The caps received by vtdec will not contain width and height, forcing a parser.
Note that adding a parser works. After a discussion, Olivier started adding
width and height parsing (from sps/pps), so they would be present in the
depayloaded caps. But might have been mistakenly added to the payloaded caps
too, that would explain the revert.

Having width and height in the depayloaded caps (read from SPS/PPS), was the
only way we found not to have to parse twice the H264 stream. From Olivier, it
seemed that it was wanted that (de)payloaders should not require a parser (for
performance reason).

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