[eel-background] drawable deprecations (GTK2), rendering-cleanup (GTK3)

1. Don't use gdk_drawable_get_display (deprecated in 2.24) 2. Use GDK_PIXMAP_XDISPLAY(pixmap) directly instead of GDK_DISPLAY_XDISPLAY(gdk_drawable_get_display(GDK_DRAWABLE (pixmap))) 3. Don't require GTK3 for stuff in GTK2.24, which we now require, and don't use gdk_drawable_get_size at all. So, use (i/o = instead of): gdk_window_get_width|height i/o gdk_drawable_get_size gdk_window_get_screen i/o gdk_drawable_get_screen gdk_window_get_visual i/o of gdk_drawable_get_visual 4. add eel_background_draw for GTK3, eel_background_expose for GTK2 This was removed in nautilus (after eel-background-box was removed): http://git.gnome.org/browse/nautilus/commit/?id=87eb53a0701252607b494f34c677b6506f82278c 5. background: port to rendering-cleanup & cope with renamings of gnome-bg APIs http://git.gnome.org/browse/nautilus/commit/?id=0b664aae4090704fa2e44d42f213da2cea8bd828 http://git.gnome.org/browse/nautilus/commit/?id=426079ef07d5f902911f30b0a9d7de95f3020374 6. Fix remaining issue #41: crashes w/ gdk_x_error BadPixmap when changing screen resolution Remove eel_background_get_pixmap_and_color(), and use the pixmap/surface & color properties in background->details directly, when needed. i.e. No more g_object/surface referencing. And call eel_background_ensure_realized() and make_color_inactive() where appropriate.
author: Jasmine Hassan <[email protected]> 2012-11-17 12:17:42 +0200
committer: Jasmine Hassan <[email protected]> 2012-11-19 07:43:02 +0200
commit: b5f9a6c811da63f5c4aca18f05cb207783a7beea (patch)
tree: 9cd226ef020732e562a1221845b4a7ad0cf1cd75 /eel
parent: dc66bc40724191f0068869773576dc4edf042c6f (diff)
download: caja-b5f9a6c811da63f5c4aca18f05cb207783a7beea.tar.bz2
caja-b5f9a6c811da63f5c4aca18f05cb207783a7beea.tar.xz
2 files changed, 162 insertions, 198 deletions
diff --git a/eel/eel-background.c b/eel/eel-background.c
index bc2be2f1..867ab2ee 100644
--- a/eel/eel-background.c
+++ b/eel/eel-background.c
@@ -30,6 +30,9 @@
 #include <gtk/gtk.h>
 #include <eel/eel-canvas.h>
 #include <eel/eel-canvas-util.h>
+#if GTK_CHECK_VERSION (3, 0, 0)
+# include <cairo-xlib.h>
+#endif
 #include <gdk/gdkx.h>
 #include <gio/gio.h>
 #include <math.h>
@@ -37,9 +40,19 @@
 #define MATE_DESKTOP_USE_UNSTABLE_API
 #include <libmateui/mate-bg.h>
 
-static GdkPixmap *eel_background_get_pixmap_and_color (EelBackground *background,
- GdkWindow *window,
- GdkColor *color);
+#if !GTK_CHECK_VERSION(3, 0, 0)
+#define cairo_surface_t GdkPixmap
+#define cairo_surface_reference g_object_ref
+#define cairo_surface_destroy g_object_unref
+#define cairo_xlib_surface_get_display GDK_PIXMAP_XDISPLAY
+#define cairo_xlib_surface_get_drawable GDK_PIXMAP_XID
+#define cairo_set_source_surface gdk_cairo_set_source_pixmap
+#define mate_bg_create_surface mate_bg_create_pixmap
+#define mate_bg_set_surface_as_root mate_bg_set_pixmap_as_root
+#define mate_bg_get_surface_from_root mate_bg_get_pixmap_from_root
+#define mate_bg_crossfade_set_start_surface mate_bg_crossfade_set_start_pixmap
+#define mate_bg_crossfade_set_end_surface mate_bg_crossfade_set_end_pixmap
+#endif
 
 static void set_image_properties (EelBackground *background);
 
@@ -56,11 +69,6 @@ enum
 LAST_SIGNAL
 };
 
-/* This is the size of the GdkRGB dither matrix, in order to avoid
- * bad dithering when tiling the gradient
- */
-#define GRADIENT_PIXMAP_TILE_SIZE 128
-
 static guint signals[LAST_SIGNAL];
 
 struct EelBackgroundDetails
@@ -71,8 +79,8 @@ struct EelBackgroundDetails
 GtkWidget *widget;
 
 /* Realized data: */
- GdkPixmap *background_pixmap;
- gboolean background_pixmap_is_unset_root_pixmap;
+ cairo_surface_t *background_surface;
+ gboolean background_surface_is_unset_root_surface;
 MateBGCrossfade *fade;
 int background_entire_width;
 int background_entire_height;
@@ -86,8 +94,8 @@ struct EelBackgroundDetails
 gulong screen_size_handler;
 /* Desktop monitors configuration watcher */
 gulong screen_monitors_handler;
- /* Can we use common pixmap for root window and desktop window */
- gboolean use_common_pixmap;
+ /* Can we use common surface for root window and desktop window */
+ gboolean use_common_surface;
 guint change_idle_id;
 
 /* activity status */
@@ -157,26 +165,24 @@ free_fade (EelBackground *background)
 }
 
 static void
-free_background_pixmap (EelBackground *background)
+free_background_surface (EelBackground *background)
 {
- GdkDisplay *display;
- GdkPixmap *pixmap;
+ cairo_surface_t *surface;
 
- pixmap = background->details->background_pixmap;
- if (pixmap != NULL)
+ surface = background->details->background_surface;
+ if (surface != NULL)
 {
- /* If we created a root pixmap and didn't set it as background
+ /* If we created a root surface and didn't set it as background
 it will live forever, so we need to kill it manually.
 If set as root background it will be killed next time the
 background is changed. */
- if (background->details->background_pixmap_is_unset_root_pixmap)
+ if (background->details->background_surface_is_unset_root_surface)
 {
- display = gdk_drawable_get_display (GDK_DRAWABLE (pixmap));
- XKillClient (GDK_DISPLAY_XDISPLAY (display),
- GDK_PIXMAP_XID (pixmap));
+ XKillClient (cairo_xlib_surface_get_display (surface),
+ cairo_xlib_surface_get_drawable (surface));
 }
- g_object_unref (pixmap);
- background->details->background_pixmap = NULL;
+ cairo_surface_destroy (surface);
+ background->details->background_surface = NULL;
 }
 }
 
@@ -254,7 +260,7 @@ eel_background_finalize (GObject *object)
 g_free (background->details->color);
 eel_background_remove_current_image (background);
 
- free_background_pixmap (background);
+ free_background_surface (background);
 
 free_fade (background);
 
@@ -313,7 +319,7 @@ eel_background_new (void)
 static void
 eel_background_unrealize (EelBackground *background)
 {
- free_background_pixmap (background);
+ free_background_surface (background);
 
 background->details->background_entire_width = 0;
 background->details->background_entire_height = 0;
@@ -324,26 +330,17 @@ eel_background_unrealize (EelBackground *background)
 
 static void
 drawable_get_adjusted_size (EelBackground *background,
- GdkDrawable *drawable,
+ GdkWindow *window,
 int *width,
 int *height)
 {
- GdkScreen *screen;
-
- /*
- * Screen resolution change makes root drawable have incorrect size.
- */
-#if GTK_CHECK_VERSION(3, 0, 0)
- width = gdk_window_get_width(GDK_WINDOW(drawable));
- height = gdk_window_get_height(GDK_WINDOW(drawable));
-#else
- gdk_drawable_get_size(drawable, width, height); // FIXME: this is right?
-#endif
-
+ *width = gdk_window_get_width(GDK_WINDOW(window));
+ *height = gdk_window_get_height(GDK_WINDOW(window));
 
 if (background->details->is_desktop)
 {
- screen = gdk_drawable_get_screen (drawable);
+ GdkScreen *screen;
+ screen = gdk_window_get_screen (window);
 *width = gdk_screen_get_width (screen);
 *height = gdk_screen_get_height (screen);
 }
@@ -387,19 +384,19 @@ eel_background_ensure_realized (EelBackground *background, GdkWindow *window)
 return FALSE;
 }
 
- free_background_pixmap (background);
+ free_background_surface (background);
 
 changed = FALSE;
 
 set_image_properties (background);
 
- background->details->background_pixmap = mate_bg_create_pixmap (background->details->bg,
- window,
- entire_width, entire_height,
- background->details->is_desktop);
- background->details->background_pixmap_is_unset_root_pixmap = background->details->is_desktop;
+ background->details->background_surface = mate_bg_create_surface (background->details->bg,
+ window,
+ entire_width, entire_height,
+ background->details->is_desktop);
+ background->details->background_surface_is_unset_root_surface = background->details->is_desktop;
 
- /* We got the pixmap and everything, so we don't care about a change
+ /* We got the surface and everything, so we don't care about a change
 that is pending (unless things actually change after this time) */
 g_object_set_data (G_OBJECT (background->details->bg),
 "ignore-pending-change", GINT_TO_POINTER (TRUE));
@@ -448,75 +445,58 @@ make_color_inactive (EelBackground *background, GdkColor *color)
 }
 }
 
-static GdkPixmap *
-eel_background_get_pixmap_and_color (EelBackground *background,
- GdkWindow *window,
- GdkColor *color)
-{
- int entire_width;
- int entire_height;
-
- drawable_get_adjusted_size (background, window, &entire_width, &entire_height);
-
- eel_background_ensure_realized (background, window);
-
- *color = background->details->default_color;
- make_color_inactive (background, color);
-
- if (background->details->background_pixmap != NULL)
- {
- return g_object_ref (background->details->background_pixmap);
- }
- return NULL;
-}
-
 void
-eel_background_expose (GtkWidget *widget,
- GdkEventExpose *event)
+#if GTK_CHECK_VERSION (3, 0, 0)
+eel_background_draw (GtkWidget *widget,
+ cairo_t *cr)
+#else
+eel_background_expose (GtkWidget *widget,
+ GdkEventExpose *event)
+#endif
 {
- GdkColor color;
 int window_width;
 int window_height;
- GdkPixmap *pixmap;
 GdkWindow *widget_window;
- cairo_t *cr;
-
- EelBackground *background;
 
 widget_window = gtk_widget_get_window (widget);
- if (event->window != widget_window)
- {
- return;
- }
 
- background = eel_get_widget_background (widget);
+#if !GTK_CHECK_VERSION (3, 0, 0)
+ g_return_if_fail (event->window != widget_window);
+#endif
+
+ EelBackground *background = eel_get_widget_background (widget);
 
 drawable_get_adjusted_size (background, widget_window, &window_width, &window_height);
 
- pixmap = eel_background_get_pixmap_and_color (background,
- widget_window,
- &color);
- cr = gdk_cairo_create (widget_window);
+ eel_background_ensure_realized (background, widget_window);
+ make_color_inactive (background, &background->details->default_color);
 
- if (pixmap) {
- gdk_cairo_set_source_pixmap (cr, pixmap, 0, 0);
+#if GTK_CHECK_VERSION(3,0,0)
+ cairo_save (cr);
+#else
+ cairo_t *cr = gdk_cairo_create (widget_window);
+#endif
+
+ if (background->details->background_surface != NULL) {
+ cairo_set_source_surface (cr, background->details->background_surface, 0, 0);
 cairo_pattern_set_extend (cairo_get_source (cr), CAIRO_EXTEND_REPEAT);
 } else {
- gdk_cairo_set_source_color (cr, &color);
+ gdk_cairo_set_source_color (cr, &background->details->default_color);
 }
 
+#if !GTK_CHECK_VERSION (3, 0, 0)
 gdk_cairo_rectangle (cr, &event->area);
 cairo_clip (cr);
+#endif
 
 cairo_rectangle (cr, 0, 0, window_width, window_height);
 cairo_fill (cr);
 
+#if GTK_CHECK_VERSION(3,0,0)
+ cairo_restore (cr);
+#else
 cairo_destroy (cr);
-
- if (pixmap)
- {
- g_object_unref (pixmap);
- }
+#endif
 }
 
 static void
@@ -703,47 +683,48 @@ eel_background_reset (EelBackground *background)
 }
 
 static void
-set_root_pixmap (EelBackground *background,
- GdkWindow *window)
+set_root_surface (EelBackground *background,
+ GdkWindow *window)
 {
- GdkPixmap *pixmap, *root_pixmap;
 GdkScreen *screen;
- GdkColor color;
 
- pixmap = eel_background_get_pixmap_and_color (background,
- window,
- &color);
- screen = gdk_drawable_get_screen (window);
+ eel_background_ensure_realized (background, window);
+ screen = gdk_window_get_screen (window);
 
- if (background->details->use_common_pixmap)
+ if (background->details->use_common_surface)
 {
- background->details->background_pixmap_is_unset_root_pixmap = FALSE;
- root_pixmap = g_object_ref (pixmap);
+ background->details->background_surface_is_unset_root_surface = FALSE;
 }
 else
 {
- root_pixmap = mate_bg_create_pixmap (background->details->bg, window,
- gdk_screen_get_width (screen), gdk_screen_get_height (screen), TRUE);
+ background->details->background_surface =
+ mate_bg_create_surface (background->details->bg, window,
+ gdk_screen_get_width (screen), gdk_screen_get_height (screen), TRUE);
 }
 
- mate_bg_set_pixmap_as_root (screen, pixmap);
+ mate_bg_set_surface_as_root (screen, background->details->background_surface);
+}
 
- g_object_unref (pixmap);
- g_object_unref (root_pixmap);
+static void
+on_fade_finished (MateBGCrossfade *fade,
+ GdkWindow *window,
+ EelBackground *background)
+{
+ set_root_surface (background, window);
 }
 
 static gboolean
-fade_to_pixmap (EelBackground *background,
- GdkWindow *window,
- GdkPixmap *pixmap)
+fade_to_surface (EelBackground *background,
+ GdkWindow *window,
+ cairo_surface_t *surface)
 {
 if (background->details->fade == NULL)
 {
 return FALSE;
 }
 
- if (!mate_bg_crossfade_set_end_pixmap (background->details->fade,
- pixmap))
+ if (!mate_bg_crossfade_set_end_surface (background->details->fade,
+ surface))
 {
 return FALSE;
 }
@@ -753,9 +734,9 @@ fade_to_pixmap (EelBackground *background,
 mate_bg_crossfade_start (background->details->fade, window);
 if (background->details->is_desktop)
 {
- g_signal_connect_swapped (background->details->fade,
- "finished",
- G_CALLBACK (set_root_pixmap), background);
+ g_signal_connect (background->details->fade,
+ "finished",
+ G_CALLBACK (on_fade_finished), background);
 }
 }
 
@@ -766,13 +747,6 @@ fade_to_pixmap (EelBackground *background,
 static void
 eel_background_set_up_widget (EelBackground *background, GtkWidget *widget)
 {
- GtkStyle *style;
- GdkPixmap *pixmap;
- GdkColor color;
-
- int window_width;
- int window_height;
-
 GdkWindow *window;
 GdkWindow *widget_window;
 gboolean in_fade;
@@ -783,13 +757,9 @@ eel_background_set_up_widget (EelBackground *background, GtkWidget *widget)
 }
 
 widget_window = gtk_widget_get_window (widget);
- drawable_get_adjusted_size (background, widget_window, &window_width, &window_height);
-
- pixmap = eel_background_get_pixmap_and_color (background,
- widget_window,
- &color);
 
- style = gtk_widget_get_style (widget);
+ eel_background_ensure_realized (background, widget_window);
+ make_color_inactive (background, &background->details->default_color);
 
 if (EEL_IS_CANVAS (widget))
 {
@@ -802,7 +772,8 @@ eel_background_set_up_widget (EelBackground *background, GtkWidget *widget)
 
 if (background->details->fade != NULL)
 {
- in_fade = fade_to_pixmap (background, window, pixmap);
+ in_fade = fade_to_surface (background, window,
+ background->details->background_surface);
 }
 else
 {
@@ -811,25 +782,23 @@ eel_background_set_up_widget (EelBackground *background, GtkWidget *widget)
 
 if (!in_fade)
 {
- if (background->details->is_desktop)
- {
- gdk_window_set_back_pixmap (window, pixmap, FALSE);
- }
- else
- {
- gdk_window_set_background (window, &color);
- gdk_window_set_back_pixmap (window, pixmap, FALSE);
+#if GTK_CHECK_VERSION (3, 0, 0)
+ cairo_pattern_t *pattern;
+ pattern = cairo_pattern_create_for_surface (background->details->background_surface);
+ gdk_window_set_background_pattern (window, pattern);
+ cairo_pattern_destroy (pattern);
+#endif
+ if (!background->details->is_desktop) {
+ gdk_window_set_background (window, &background->details->default_color);
 }
+#if !GTK_CHECK_VERSION (3, 0, 0)
+ gdk_window_set_back_pixmap (window, background->details->background_surface, FALSE);
+#endif
 }
 
 if (background->details->is_desktop && !in_fade)
 {
- set_root_pixmap (background, window);
- }
-
- if (pixmap)
- {
- g_object_unref (pixmap);
+ set_root_surface (background, window);
 }
 }
 
@@ -871,14 +840,8 @@ init_fade (EelBackground *background, GtkWidget *widget)
 * we don't want to crossfade
 */
 window = gtk_widget_get_window (widget);
-
-#if GTK_CHECK_VERSION(3, 0, 0)
 old_width = gdk_window_get_width(GDK_WINDOW(window));
 old_height = gdk_window_get_height(GDK_WINDOW(window));
-#else
- gdk_drawable_get_size(window, &old_width, &old_height);
-#endif
-
 drawable_get_adjusted_size (background, window,
 &width, &height);
 if (old_width == width && old_height == height)
@@ -893,19 +856,19 @@ init_fade (EelBackground *background, GtkWidget *widget)
 
 if (background->details->fade != NULL && !mate_bg_crossfade_is_started (background->details->fade))
 {
- GdkPixmap *start_pixmap;
+ cairo_surface_t *start_surface;
 
- if (background->details->background_pixmap == NULL)
+ if (background->details->background_surface == NULL)
 {
- start_pixmap = mate_bg_get_pixmap_from_root (gtk_widget_get_screen (widget));
+ start_surface = mate_bg_get_surface_from_root (gtk_widget_get_screen (widget));
 }
 else
 {
- start_pixmap = g_object_ref (background->details->background_pixmap);
+ start_surface = cairo_surface_reference (background->details->background_surface);
 }
- mate_bg_crossfade_set_start_pixmap (background->details->fade,
- start_pixmap);
- g_object_unref (start_pixmap);
+ mate_bg_crossfade_set_start_surface (background->details->fade,
+ start_surface);
+ cairo_surface_destroy (start_surface);
 }
 }
 
@@ -980,17 +943,13 @@ widget_realized_setup (GtkWidget *widget, gpointer data)
 
 root_window = gdk_screen_get_root_window(screen);
 
-#if GTK_CHECK_VERSION(3, 0, 0)
 if (gdk_window_get_visual(root_window) == gtk_widget_get_visual(widget))
-#else
- if (gdk_drawable_get_visual(root_window) == gtk_widget_get_visual(widget))
-#endif
 {
- background->details->use_common_pixmap = TRUE;
+ background->details->use_common_surface = TRUE;
 }
 else
 {
- background->details->use_common_pixmap = FALSE;
+ background->details->use_common_surface = FALSE;
 }
 
 init_fade (background, widget);
@@ -1028,7 +987,7 @@ widget_unrealize_cb (GtkWidget *widget, gpointer data)
 background->details->screen_monitors_handler);
 background->details->screen_monitors_handler = 0;
 }
- background->details->use_common_pixmap = FALSE;
+ background->details->use_common_surface = FALSE;
 }
 
 void
diff --git a/eel/eel-background.h b/eel/eel-background.h
index cfccf2a1..1c2218f3 100644
--- a/eel/eel-background.h
+++ b/eel/eel-background.h
@@ -75,54 +75,59 @@ EelBackground * eel_background_new (voi
 
 
 /* Calls to change a background. */
-void eel_background_set_use_base (EelBackground *background,
- gboolean use_base);
-void eel_background_set_color (EelBackground *background,
- const char *color_or_gradient);
-void eel_background_set_image_uri (EelBackground *background,
- const char *image_uri);
+void eel_background_set_use_base (EelBackground *background,
+ gboolean use_base);
+void eel_background_set_color (EelBackground *background,
+ const char *color_or_gradient);
+void eel_background_set_image_uri (EelBackground *background,
+ const char *image_uri);
 
-void eel_background_reset (EelBackground *background);
-void eel_background_set_image_placement (EelBackground *background,
- EelBackgroundImagePlacement placement);
+void eel_background_reset (EelBackground *background);
+void eel_background_set_image_placement (EelBackground *background,
+ EelBackgroundImagePlacement placement);
 
 /* Should be TRUE for desktop background */
-void eel_background_set_desktop (EelBackground *background,
- GtkWidget *widget,
- gboolean is_desktop);
-gboolean eel_background_is_desktop (EelBackground *background);
+void eel_background_set_desktop (EelBackground *background,
+ GtkWidget *widget,
+ gboolean is_desktop);
+gboolean eel_background_is_desktop (EelBackground *background);
 
 /* Calls to interrogate the current state of a background. */
-char * eel_background_get_color (EelBackground *background);
-char * eel_background_get_image_uri (EelBackground *background);
-EelBackgroundImagePlacement eel_background_get_image_placement (EelBackground *background);
-gboolean eel_background_is_dark (EelBackground *background);
-gboolean eel_background_is_set (EelBackground *background);
+char * eel_background_get_color (EelBackground *background);
+char * eel_background_get_image_uri (EelBackground *background);
+EelBackgroundImagePlacement eel_background_get_image_placement (EelBackground *background);
+gboolean eel_background_is_dark (EelBackground *background);
+gboolean eel_background_is_set (EelBackground *background);
 
 /* Helper function for widgets using EelBackground */
-void eel_background_expose (GtkWidget *widget,
- GdkEventExpose *event);
+#if GTK_CHECK_VERSION (3, 0, 0)
+void eel_background_draw (GtkWidget *widget,
+ cairo_t *cr)
+#else
+void eel_background_expose (GtkWidget *widget,
+ GdkEventExpose *event);
+#endif
 
 /* Handles a dragged color being dropped on a widget to change the background color. */
-void eel_background_receive_dropped_color (EelBackground *background,
- GtkWidget *widget,
- GdkDragAction action,
- int drop_location_x,
- int drop_location_y,
- const GtkSelectionData *dropped_color);
+void eel_background_receive_dropped_color (EelBackground *background,
+ GtkWidget *widget,
+ GdkDragAction action,
+ int drop_location_x,
+ int drop_location_y,
+ const GtkSelectionData *dropped_color);
 
 /* Handles a special-case image name that means "reset to default background" too. */
-void eel_background_receive_dropped_background_image (EelBackground *background,
- GdkDragAction action,
- const char *image_uri);
+void eel_background_receive_dropped_background_image (EelBackground *background,
+ GdkDragAction action,
+ const char *image_uri);
 
 /* Gets or creates a background so that it's attached to a widget. */
-EelBackground * eel_get_widget_background (GtkWidget *widget);
-void eel_background_save_to_settings (EelBackground *background);
+EelBackground * eel_get_widget_background (GtkWidget *widget);
+void eel_background_save_to_settings (EelBackground *background);
 
 /* Set activity status of background. Inactive backgrounds are drawn in the theme's INSENSITIVE color. */
-void eel_background_set_active (EelBackground *background,
- gboolean is_active);
+void eel_background_set_active (EelBackground *background,
+ gboolean is_active);
 
 typedef struct EelBackgroundDetails EelBackgroundDetails;
author	Jasmine Hassan <[email protected]>	2012-11-17 12:17:42 +0200
committer	Jasmine Hassan <[email protected]>	2012-11-19 07:43:02 +0200
commit	b5f9a6c811da63f5c4aca18f05cb207783a7beea (patch)
tree	9cd226ef020732e562a1221845b4a7ad0cf1cd75 /eel
parent	dc66bc40724191f0068869773576dc4edf042c6f (diff)
download	caja-b5f9a6c811da63f5c4aca18f05cb207783a7beea.tar.bz2 caja-b5f9a6c811da63f5c4aca18f05cb207783a7beea.tar.xz