[Bug 752690] h263parse: update proper PAR calculations

[GStreamer (GNOME Bugzilla)]

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

--- Comment #10 from Vineeth <vineeth.tm at samsung.com> ---
maybe my last message was confusing.. let me try to clarify here...

it can be reproduced with any h263 stream..
i used one from http://download.wavetlan.com/SVV/Media/HTTP/http-3gp.htm

so what happens here is 
in avdec_h263 pad, PAR is 12/11. This follows the logic (prefer non-1/1 PAR
from bitstream if demuxer PAR is 1/1) as written in gstavviddec.c
in h263parse pad, PAR is 1/1. This happens because, with previous logic,
h263parse checks its sink caps and if PAR is already present there, uses it by
default.
Here qtdemux calculates it to 1/1 and sends it across to h263parse.

0:00:00.819054604 29351 0x7f1060008800 DEBUG               GST_CAPS
gstpad.c:2574:gst_pad_get_current_caps:<h263parse0:src> get current pad caps
video/x-h263, variant=(string)itu, width=(int)176, height=(int)144,
framerate=(fraction)5000/6689, pixel-aspect-ratio=(fraction)1/1,
parsed=(boolean)true, annex-d=(boolean)false, annex-e=(boolean)false,
annex-f=(boolean)false, annex-g=(boolean)false, annex-i=(boolean)false,
annex-j=(boolean)false, annex-k=(boolean)false, annex-m=(boolean)false,
annex-n=(boolean)false, annex-q=(boolean)false, annex-r=(boolean)false,
annex-s=(boolean)false, annex-t=(boolean)false, annex-u=(boolean)false,
annex-v=(boolean)false, profile=(string)0, level=(string)10
0:00:00.819139260 29351 0x7f1060008800 DEBUG               GST_CAPS
gstpad.c:2574:gst_pad_get_current_caps:<qtdemux0:video_0> get current pad caps
video/x-h263, variant=(string)itu, width=(int)176, height=(int)144,
framerate=(fraction)5000/6689, pixel-aspect-ratio=(fraction)1/1

0:00:00.843486064 29351 0x7f1060008800 DEBUG               GST_CAPS
gstpad.c:2574:gst_pad_get_current_caps:<avdec_h263-0:src> get current pad caps
video/x-raw, format=(string)I420, width=(int)176, height=(int)144,
interlace-mode=(string)progressive, pixel-aspect-ratio=(fraction)12/11,
chroma-site=(string)jpeg, colorimetry=(string)bt601,
framerate=(fraction)5000/6689

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