GNOME Bugzilla – Bug 788206
Background screenshot code doesn't work in GTK+ 3.22
Last modified: 2017-11-01 16:40:55 UTC
Created attachment 360480 [details] [review] background: Drop dead background screenshot code cc-background-panel.c contains: gdk_screen_get_monitor_geometry (gtk_widget_get_screen (widget), primary, &data->monitor_rect); gdk_screen_get_monitor_workarea (gtk_widget_get_screen (widget), primary, &data->workarea_rect); Which both get the same result due to the following changed in GTK 3.22 void gdk_screen_get_monitor_workarea (GdkScreen *screen, gint monitor_num, GdkRectangle *dest) { ... /* FIXME */ gdk_monitor_get_geometry (monitor, dest); } Due to these values being the same a screenshot of zero size is requested. This causes a slight delay showing the background, but doesn't change the content. This may be a bug in GTK+, or this screenshot code is not required anymore. I've attached a patch that removes it if we don't need it anymore. (Bug was found migrating to non-deprecated API which then did get the correct workarea and the background looked broken).
See bug 788208 for GTK+ regression.
Review of attachment 360480 [details] [review]: this looks good. the screenshot was only trying to capture gnome-shell's top bar to overlay on the background preview but now on 3.26 the top bar might be transparent which obviously wouldn't work at all for previewing a different wallpaper so that's another reason to drop this
*** Bug 789624 has been marked as a duplicate of this bug. ***
*** Bug 782415 has been marked as a duplicate of this bug. ***
Attachment 360480 [details] pushed as 6723ff4 - background: Drop dead background screenshot code