eel: Avoid deprecated gdk_window_set_background functions

Fixes partially #506, fixes compiz-reloaded/compiz#40.
author: Alexei Sorokin <[email protected]> 2017-03-12 15:07:16 +0300
committer: raveit65 <[email protected]> 2017-03-13 10:28:08 +0100
commit: 4c10ec0fdac58498c8c9616b940ff78990c86c90 (patch)
tree: 7cd24f21ef9f150fd7a565e255e3ea2e7dafb8c8 /eel/eel-background.c
parent: 438682bef1a8fea49b4c6972350bede1bfad69d0 (diff)
download: caja-4c10ec0fdac58498c8c9616b940ff78990c86c90.tar.bz2
caja-4c10ec0fdac58498c8c9616b940ff78990c86c90.tar.xz
1 files changed, 117 insertions, 45 deletions
diff --git a/eel/eel-background.c b/eel/eel-background.c
index c1ef5c6c..76f47a69 100644
--- a/eel/eel-background.c
+++ b/eel/eel-background.c
@@ -53,6 +53,7 @@ static guint signals[LAST_SIGNAL] = { 0 };
 struct EelBackgroundDetails
 {
 GtkWidget *widget;
+ GtkWidget *front_widget;
 MateBG *bg;
 char *color;
 
@@ -79,6 +80,10 @@ struct EelBackgroundDetails
 gboolean is_active;
 };
 
+#if GTK_CHECK_VERSION (3, 22, 0)
+static GList *desktop_bg_objects = NULL;
+#endif
+
 static void
 free_fade (EelBackground *self)
 {
@@ -124,9 +129,16 @@ eel_background_finalize (GObject *object)
 }
 
 free_background_surface (self);
-
 free_fade (self);
 
+#if GTK_CHECK_VERSION (3, 22, 0)
+ if (self->details->is_desktop)
+ {
+ desktop_bg_objects = g_list_remove (desktop_bg_objects,
+ G_OBJECT (self));
+ }
+
+#endif
 G_OBJECT_CLASS (eel_background_parent_class)->finalize (object);
 }
 
@@ -358,11 +370,15 @@ void
 eel_background_draw (GtkWidget *widget,
 cairo_t *cr)
 {
- int width, height;
- GdkWindow *window = gtk_widget_get_window (widget);
+ EelBackground *self = eel_get_widget_background (widget);
 GdkRGBA color;
+ int width, height;
 
- EelBackground *self = eel_get_widget_background (widget);
+ if (self->details->fade != NULL &&
+ mate_bg_crossfade_is_started (self->details->fade))
+ {
+ return;
+ }
 
 drawable_get_adjusted_size (self, &width, &height);
 
@@ -372,7 +388,8 @@ eel_background_draw (GtkWidget *widget,
 
 cairo_save (cr);
 
- if (self->details->bg_surface != NULL) {
+ if (self->details->bg_surface != NULL)
+ {
 cairo_set_source_surface (cr, self->details->bg_surface, 0, 0);
 cairo_pattern_set_extend (cairo_get_source (cr), CAIRO_EXTEND_REPEAT);
 } else {
@@ -468,7 +485,7 @@ on_fade_finished (MateBGCrossfade *fade,
 
 static gboolean
 fade_to_surface (EelBackground *self,
- GdkWindow *window,
+ GtkWidget *widget,
 cairo_surface_t *surface)
 {
 if (self->details->fade == NULL ||
@@ -479,7 +496,22 @@ fade_to_surface (EelBackground *self,
 
 if (!mate_bg_crossfade_is_started (self->details->fade))
 {
+#if GTK_CHECK_VERSION (3, 22, 0)
+ mate_bg_crossfade_start_widget (self->details->fade, widget);
+#else
+ GdkWindow *window;
+
+ if (EEL_IS_CANVAS (widget))
+ {
+ window = gtk_layout_get_bin_window (GTK_LAYOUT (widget));
+ }
+ else
+ {
+ window = gtk_widget_get_window (widget);
+ }
+
 mate_bg_crossfade_start (self->details->fade, window);
+#endif
 if (self->details->is_desktop)
 {
 g_signal_connect (self->details->fade,
@@ -494,9 +526,7 @@ fade_to_surface (EelBackground *self,
 static void
 eel_background_set_up_widget (EelBackground *self)
 {
- GdkWindow *window;
 GtkWidget *widget = self->details->widget;
- GdkRGBA color;
 
 gboolean in_fade = FALSE;
 
@@ -504,41 +534,48 @@ eel_background_set_up_widget (EelBackground *self)
 return;
 
 eel_background_ensure_realized (self);
- color = self->details->default_color;
- make_color_inactive (self, &color);
 
 if (self->details->bg_surface == NULL)
 return;
 
- if (EEL_IS_CANVAS (widget)) {
- window = gtk_layout_get_bin_window (GTK_LAYOUT (widget));
- } else {
- window = gtk_widget_get_window (widget);
- }
+ gtk_widget_queue_draw (widget);
 
 if (self->details->fade != NULL)
- in_fade = fade_to_surface (self, window, self->details->bg_surface);
+ in_fade = fade_to_surface (self, widget, self->details->bg_surface);
 
 if (!in_fade)
 {
- cairo_pattern_t *pattern;
- pattern = cairo_pattern_create_for_surface (self->details->bg_surface);
- gdk_window_set_background_pattern (window, pattern);
- cairo_pattern_destroy (pattern);
-
+ GdkWindow *window;
 
- if (self->details->is_desktop)
+ if (EEL_IS_CANVAS (widget))
 {
- set_root_surface (self, window, gtk_widget_get_screen (widget));
+ window = gtk_layout_get_bin_window (GTK_LAYOUT (widget));
 }
 else
 {
-
- gdk_window_set_background_rgba (window, &color);
-
+ window = gtk_widget_get_window (widget);
 }
 
- gdk_window_invalidate_rect (window, NULL, TRUE);
+ if (self->details->is_desktop)
+ {
+#if !GTK_CHECK_VERSION (3, 22, 0)
+ if (self->details->bg_surface != NULL)
+ {
+ cairo_pattern_t *pattern =
+ cairo_pattern_create_for_surface (self->details->bg_surface);
+ gdk_window_set_background_pattern (window, pattern);
+ cairo_pattern_destroy (pattern);
+ }
+ else
+ {
+ GdkRGBA color = self->details->default_color;
+ make_color_inactive (self, &color);
+ gdk_window_set_background_rgba (window, &color);
+ }
+ gdk_window_invalidate_rect (window, NULL, TRUE);
+#endif
+ set_root_surface (self, window, gtk_widget_get_screen (widget));
+ }
 }
 }
 
@@ -553,8 +590,6 @@ background_changed_cb (EelBackground *self)
 eel_background_unrealize (self);
 eel_background_set_up_widget (self);
 
- gtk_widget_queue_draw (self->details->widget);
-
 return FALSE;
 }
 
@@ -577,12 +612,10 @@ widget_queue_background_change (GtkWidget *widget,
 * EelBackgroundStyle so that it will match the chosen GTK+ theme.
 */
 static void
-widget_style_set_cb (GtkWidget *widget,
- GtkStyle *previous_style,
- gpointer user_data)
+widget_style_updated_cb (GtkWidget *widget,
+ gpointer user_data)
 {
- if (previous_style != NULL)
- widget_queue_background_change (widget, user_data);
+ widget_queue_background_change (widget, user_data);
 }
 
 static void
@@ -692,7 +725,9 @@ on_widget_destroyed (GtkWidget *widget,
 }
 
 free_fade (self);
+
 self->details->widget = NULL;
+ self->details->front_widget = NULL;
 }
 
 /* Gets the background attached to a widget.
@@ -701,9 +736,8 @@ on_widget_destroyed (GtkWidget *widget,
 this will create one. To change the widget's background, you can
 just call eel_background methods on the widget.
 
- If the widget is a canvas, nothing more needs to be done. For
- normal widgets, you need to call eel_background_draw() from your
- draw/expose handler to draw the background.
+ You need to call eel_background_draw() from your draw event handler to
+ draw the background.
 
 Later, we might want a call to find out if we already have a background,
 or a way to share the same background among multiple widgets; both would
@@ -712,22 +746,41 @@ on_widget_destroyed (GtkWidget *widget,
 EelBackground *
 eel_get_widget_background (GtkWidget *widget)
 {
+ EelBackground *self;
+ gpointer data;
+#if GTK_CHECK_VERSION (3, 22, 0)
+ GList *l;
+#endif
 
 g_return_val_if_fail (GTK_IS_WIDGET (widget), NULL);
 
 /* Check for an existing background. */
- gpointer data = g_object_get_data (G_OBJECT (widget), "eel_background");
+ data = g_object_get_data (G_OBJECT (widget), "eel_background");
 if (data != NULL)
 {
 g_assert (EEL_IS_BACKGROUND (data));
 return data;
 }
+#if GTK_CHECK_VERSION (3, 22, 0)
+ /* Check for an existing desktop window background. */
+ for (l = desktop_bg_objects; l != NULL; l = l->next)
+ {
+ g_assert (EEL_IS_BACKGROUND (l->data));
+ self = EEL_BACKGROUND (l->data);
+ if (widget == self->details->widget)
+ {
+ return self;
+ }
+ }
+#endif
+
+ self = eel_background_new ();
+ self->details->widget = widget;
+ self->details->front_widget = widget;
 
 /* Store the background in the widget's data. */
- EelBackground *self = eel_background_new ();
 g_object_set_data_full (G_OBJECT (widget), "eel_background",
 self, g_object_unref);
- self->details->widget = widget;
 
 g_signal_connect_object (widget, "destroy",
 G_CALLBACK (on_widget_destroyed), self, 0);
@@ -736,8 +789,8 @@ eel_get_widget_background (GtkWidget *widget)
 g_signal_connect_object (widget, "unrealize",
 G_CALLBACK (widget_unrealize_cb), self, 0);
 
- g_signal_connect_object (widget, "style_set",
- G_CALLBACK (widget_style_set_cb), self, 0);
+ g_signal_connect_object (widget, "style-updated",
+ G_CALLBACK (widget_style_updated_cb), self, 0);
 
 /* Arrange to get the signal whenever the background changes. */
 g_signal_connect_object (self, "appearance_changed",
@@ -828,14 +881,32 @@ eel_background_reset (EelBackground *self)
 
 void
 eel_background_set_desktop (EelBackground *self,
- GtkWidget *widget,
 gboolean is_desktop)
 {
 self->details->is_desktop = is_desktop;
 
- if (gtk_widget_get_realized (widget) && is_desktop)
+ if (is_desktop)
 {
- widget_realized_setup (widget, self);
+#if GTK_CHECK_VERSION (3, 22, 0)
+ self->details->widget =
+ gtk_widget_get_toplevel (self->details->front_widget);
+
+ desktop_bg_objects = g_list_prepend (desktop_bg_objects,
+ G_OBJECT (self));
+
+#endif
+ if (gtk_widget_get_realized (self->details->widget))
+ {
+ widget_realized_setup (self->details->widget, self);
+ }
+ }
+ else
+ {
+#if GTK_CHECK_VERSION (3, 22, 0)
+ desktop_bg_objects = g_list_remove (desktop_bg_objects,
+ G_OBJECT (self));
+ self->details->widget = self->details->front_widget;
+#endif
 }
 }
 
@@ -853,6 +924,7 @@ eel_background_set_active (EelBackground *self,
 {
 self->details->is_active = is_active;
 set_image_properties (self);
+ gtk_widget_queue_draw (self->details->widget);
 }
 }
author	Alexei Sorokin <[email protected]>	2017-03-12 15:07:16 +0300
committer	raveit65 <[email protected]>	2017-03-13 10:28:08 +0100
commit	4c10ec0fdac58498c8c9616b940ff78990c86c90 (patch)
tree	7cd24f21ef9f150fd7a565e255e3ea2e7dafb8c8 /eel/eel-background.c
parent	438682bef1a8fea49b4c6972350bede1bfad69d0 (diff)
download	caja-4c10ec0fdac58498c8c9616b940ff78990c86c90.tar.bz2 caja-4c10ec0fdac58498c8c9616b940ff78990c86c90.tar.xz