[Bug 741263] videodecoder: implement caps query

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=741263
  GStreamer | gst-plugins-base | unspecified

Thiago Sousa Santos <thiagossantos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|HEAD                        |1.5.1

--- Comment #23 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-12-17 22:49:02 UTC ---
Thanks for the reviews. Pushed for video and also for audio.

commit 8085352fb32c293c1128f5fcc6646f1f5e74663c
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Wed Dec 17 14:18:03 2014 -0300

    videodecoder: expose getcaps virtual function

    Allows subclasses to do custom caps query replies.

    Also exposes the standard caps query handler so subclasses can just
    extend on top of it instead of reimplementing the caps query proxying.

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

commit 49568005494bfdaf82734437ccaa0584eab22c9c
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Mon Dec 15 18:46:21 2014 -0300

    videodecoder: accept-caps should only require fields from the template

    With the new caps query results the caps returned might have extra fields
    that are not required by the decoder (framerate for image decoders) and it
    causes a regression making, for example, jpegdec reject caps that don't
    have framerates.

    The accept-caps implementation will do 2 checks:

    1) Do subset check with the template caps, making sure all the required
    fields that are present on the template are present on the received caps.
    2) Do a intersection check with the result of a caps query, making sure
    that downstream can accept the fields in the received caps.

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

commit 00c2ce60c34634008e533c96bcb5423cdbd32c41
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Tue Dec 9 16:08:12 2014 -0300

    videoutils: proxy filter when doing a caps query downstream

    Allows downstream to use the filter and possibly reduce caps complexity
    to speed up negotiation

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

commit f49208555226d3f65816ed143440a1777551620f
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Tue Dec 9 16:05:27 2014 -0300

    videoutils: return empty if the element has no possible allowed caps

    Instead of returning the template caps and having a failure happen
    later because there are no possible caps

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

commit f24075887f9d876af59f27a389f2e8c7d6c957ca
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Mon Dec 8 16:33:33 2014 -0300

    videodecoder: implement caps query

    Refactor the encoder's caps query proxying function to a common place
    and use it in the videodecoder to proxy downstream restrictions.

    The new function is private to the gstvideo lib.

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

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