GNOME Bugzilla – Bug 673137
Grab the current window -> Screenshot is mirrored
Last modified: 2013-07-05 14:47:30 UTC
Created attachment 210920 [details] Grab the current window Grab the current window -> Taken screenshot reverses. 1. alt + prtscr or $ gnome-screenshot -i gnome-screenshot-3.4.0-1.fc17.x86_64
Bizarre; however, the code that writes out the screenshot lives in gnome-shell, so lets move this bug there.
What version of gnome-shell are you using?
(In reply to comment #2) > What version of gnome-shell are you using? $ rpm -q gnome-shell gnome-shell-3.4.0-1.fc17.x86_64
*** Bug 673227 has been marked as a duplicate of this bug. ***
Cogl-WARNING **: ./cogl-framebuffer.c:1334: GL error (1282): Invalid operation Cogl-WARNING **: ./cogl-framebuffer.c:1334: GL error (1282): Invalid operation Seems like bug 669368 is back ...
Created attachment 211029 [details] [review] texture: Fix error handling in get_texture_bits_via_offscreen get_texture_bits_via_offscreen does not check the return value of cogl_framebuffer_read_pixels_into_bitmap which results into never using the fallback path texture_get_cb. cogl_framebuffer_read_pixels_into_bitmap does not check whether the framebuffer is properly allocated though; so fix that as well.
It looks like I accidentally took out the call to cogl_framebuffer_allocate from get_texture_bits_via_offscreen in 10a38bb14fac3. Sorry about that. I guess we might want to add it back in to bail out sooner. The patch looks good to land anyway though so I've pushed it to master and the cogl-1.10 branch. Thanks for fixing this again!
Hi, I'm trying to port this patch to Ubuntu's precise that still ships a cogl without it. However, after applying the patch and recompiling, I get a black image instead of the correctly reversed image. Does this patch depend on some other previous patch? -- Regards, Marga
(In reply to comment #8) > Hi, > > I'm trying to port this patch to Ubuntu's precise that still ships a cogl > without it. However, after applying the patch and recompiling, I get a black > image instead of the correctly reversed image. Does this patch depend on some > other previous patch? Which GPU / driver are you using?
Hi, It's an nvidia card with the proprietary driver. I also tested the quantal version and I got a black screen. I found a number of bugs reporting that but for virtual machines, so apparently this particular fix is bad for some cards? -- Regards, Marga
(In reply to comment #10) > Hi, > > It's an nvidia card with the proprietary driver. I also tested the quantal > version and I got a black screen. I found a number of bugs reporting that but > for virtual machines, so apparently this particular fix is bad for some cards? https://bugzilla.gnome.org/show_bug.cgi?id=696094#c7
But how did I trigger it just by adding the call to framebuffer allocate and the return value checking?
I've tested the same versions on a machine with intel graphics card and the screenshot was not flipped. So, yes, it's an nvidia issue. But the fix reported here makes it worse, not better.