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 754557 - Camera output is all green since clutter-gst-3.0 port
Camera output is all green since clutter-gst-3.0 port
Status: RESOLVED FIXED
Product: clutter-gst
Classification: Other
Component: general
3.0.x
Other Linux
: Normal normal
: ---
Assigned To: clutter-gst-maint
clutter-gst-maint
Depends on:
Blocks:
 
 
Reported: 2015-09-04 08:38 UTC by Bastien Nocera
Modified: 2015-09-04 11:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
3.16 (434.67 KB, image/png)
2015-09-04 08:40 UTC, Bastien Nocera 		  Details
3.17 (138.45 KB, image/png)
2015-09-04 08:42 UTC, Bastien Nocera 		  Details
video-sink: fix CoglError leak (984 bytes, patch)
2015-09-04 11:48 UTC, Lionel Landwerlin 		committed Details | Review
video-sink: set pixel format to premultiplied for ayuv (1.12 KB, patch)
2015-09-04 11:48 UTC, Lionel Landwerlin 		committed Details | Review

Description Bastien Nocera 2015-09-04 08:38:47 UTC 
Will attach test images.
Comment 1 Bastien Nocera 2015-09-04 08:40:15 UTC 
Created attachment 310643 [details]
3.16
Comment 2 Bastien Nocera 2015-09-04 08:42:27 UTC 
Created attachment 310645 [details]
3.17
Comment 3 Bastien Nocera 2015-09-04 09:46:10 UTC 
Apparently also happens with totem, so that'd be a clutter-gst bug:
http://i.imgur.com/jMdCb2n.png
Comment 4 Lionel Landwerlin 2015-09-04 11:48:54 UTC 
Created attachment 310659 [details] [review]
video-sink: fix CoglError leak
Comment 5 Lionel Landwerlin 2015-09-04 11:48:58 UTC 
Created attachment 310660 [details] [review]
video-sink: set pixel format to premultiplied for ayuv

We do premultiply in the fragment shader for the ayuv pixel format, so
set the cogl pixel format to premultiplied so Cogl/Mesa don't do any
work on the data.
Comment 6 Lionel Landwerlin 2015-09-04 11:57:15 UTC 
Review of attachment 310659 [details] [review]:

Pushed to 3.0 branch
Comment 7 Lionel Landwerlin 2015-09-04 11:57:29 UTC 
Review of attachment 310660 [details] [review]:

Pushed to 3.0 branch.
Comment 8 Lionel Landwerlin 2015-09-04 11:57:56 UTC 
Thanks for the report.
Fixed and made a 3.0.10 release.