diff options
author | Carlos Garnacho <carlosg@gnome.org> | 2018-03-09 16:46:59 +0100 |
---|---|---|
committer | Florian Müllner <fmuellner@gnome.org> | 2018-03-21 20:37:34 +0100 |
commit | 44a7f74dcd8a42ec5144991e412a79406ff16430 (patch) | |
tree | 7644befd0ea4d22d0123d18803a25762e3e274ec /cogl | |
parent | 3832c6b607f92ae7043db60d9e69f5652fd97d70 (diff) | |
download | mutter-44a7f74dcd8a42ec5144991e412a79406ff16430.tar.gz |
cogl: Read pixels as per the stored format
By the looks of it, commit 95e9fa10ef was taping over an Intel DRI bug
that would make it return post-swizzling pixel data on glReadPixels().
There's been reports over time of that commit resulting in wrong colors
on other drivers, and lately Mesa >17.3 started showing the same symptoms
on Intel.
But texture swizzling works by changing parameters before fragment shaders
and reading pixels from an already drawn FBO/texture doesn't involve those.
This should thus use pixel_format_to_gl_with_target(), which will result in
correctly requesting the same pixel format than the underlying texture,
while still considering it BGRA for the upper layers in the swizzling case.
https://gitlab.gnome.org/GNOME/mutter/issues/72
Closes: #72
Diffstat (limited to 'cogl')
-rw-r--r-- | cogl/cogl/driver/gl/cogl-framebuffer-gl.c | 22 |
1 files changed, 6 insertions, 16 deletions
diff --git a/cogl/cogl/driver/gl/cogl-framebuffer-gl.c b/cogl/cogl/driver/gl/cogl-framebuffer-gl.c index 8ae631df6..abff84ee0 100644 --- a/cogl/cogl/driver/gl/cogl-framebuffer-gl.c +++ b/cogl/cogl/driver/gl/cogl-framebuffer-gl.c @@ -1412,22 +1412,12 @@ _cogl_framebuffer_gl_read_pixels_into_bitmap (CoglFramebuffer *framebuffer, if (!cogl_is_offscreen (framebuffer)) y = framebuffer_height - y - height; - required_format = ctx->driver_vtable->pixel_format_to_gl (ctx, - format, - &gl_intformat, - &gl_format, - &gl_type); -#if HAVE_COGL_GL - /* As we are reading pixels, we want to consider the bitmap according to - * its real pixel format, not the swizzled channels we pretend face to the - * pipeline. - */ - if ((ctx->driver == COGL_DRIVER_GL || ctx->driver == COGL_DRIVER_GL3) && - (format == COGL_PIXEL_FORMAT_BGRA_8888 || - format == COGL_PIXEL_FORMAT_BGRA_8888_PRE) && - _cogl_has_private_feature (ctx, COGL_PRIVATE_FEATURE_TEXTURE_SWIZZLE)) - gl_format = GL_BGRA; -#endif + required_format = ctx->driver_vtable->pixel_format_to_gl_with_target (ctx, + framebuffer->internal_format, + format, + &gl_intformat, + &gl_format, + &gl_type); /* NB: All offscreen rendering is done upside down so there is no need * to flip in this case... */ |