[Bug 723289] New: cairooverlay: add RGB16 support

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=723289
  GStreamer | gst-plugins-good | 1.x

           Summary: cairooverlay: add RGB16 support
    Classification: Platform
           Product: GStreamer
           Version: 1.x
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: julien.isorce at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=267646)
 View: https://bugzilla.gnome.org/attachment.cgi?id=267646
 Review: https://bugzilla.gnome.org/review?bug=723289&attachment=267646

cairooverlay: add support for RGB16

* actual result:

On RPI it's currently not possible to do gst-launch-1.0 videotestsrc !
"video/x-raw, format=RGB16" ! cairooverlay ! "video/x-raw, format=RGB16" !
ximagesink

and ximagesink reports to support GST_VIDEO_FORMAT_RGB16

(It's intended to work with gst-omx+resize component but easier to report
something with videotestsrc)

* other infos:

In the following patch that's a bit weird to have:

#if G_BYTE_ORDER == G_LITTLE_ENDIAN
#define TEMPLATE_CAPS GST_VIDEO_CAPS_MAKE("{ RGB16, BGRx, BGRA }")
#else
#define TEMPLATE_CAPS GST_VIDEO_CAPS_MAKE("{ BGR16, xRGB, ARGB }")


Shouldn't be BGR16 on the first line and RGB16 on the second ? But ximagesink
report RGB16 and videotestsrc ! ximagesink  shows the right color. (if
ximagesink reports wrong then that would mean videotestsrc generates the wrong
image)

Let me know what you think. Also format like RGBx and xBGR have no equivalent
in cairo ?

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