After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 704009 - mpegvideoparse: Use sequence_display_extension values to update the src caps
mpegvideoparse: Use sequence_display_extension values to update the src caps
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.1.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-11 15:03 UTC by sreerenj
Modified: 2013-07-12 12:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
mpegvideoparse: Use sequence_display_extension values to update the src caps (2.01 KB, patch)
2013-07-11 15:03 UTC, sreerenj 		committed Details | Review

Description sreerenj 2013-07-11 15:03:57 UTC 
Created attachment 248931 [details] [review]
mpegvideoparse: Use sequence_display_extension values to update the src caps

The caps should always represent what the user is supposed to see.So if there is a sequence_display_extension associated with the stream then use the   display_horizontal_size/display_vertical_size to update the src caps (iff they are less than the values provided by sequence header).

reference: https://bugzilla.gnome.org/show_bug.cgi?id=694068
Comment 1 Sebastian Dröge (slomo) 2013-07-12 08:06:48 UTC 
commit 654744a03c807b6c1f7ece6d8316a899237fda11
Author: Sreerenj Balachandran <sreerenj.balachandran@intel.com>
Date:   Thu Jul 11 17:46:37 2013 +0300

    mpegvideoparse: Use sequence_display_extension values to update the src caps
    
    The caps should always represent what the user is supposed to see.
    So if there is a sequence_display_extension associated with the
    stream then use the display_horizontal_size/display_vertical_size
    to update the src caps (if they are less than the values provided
    by sequence header).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=704009
Comment 2 Sebastian Dröge (slomo) 2013-07-12 08:07:18 UTC 
Is something like this also needed in h263parse and mpeg4videoparse?
Comment 3 sreerenj 2013-07-12 12:36:31 UTC 
Possible..I am not sure whether the codecparser for mpeg4_2 is parsing all the necessary stuffs for this.