773 lines
25 KiB
Diff
773 lines
25 KiB
Diff
From 53d505622e90805684260873d0fbadaec9e2d3ed Mon Sep 17 00:00:00 2001
|
|
From: Victor Kareh <vkareh@redhat.com>
|
|
Date: Mon, 20 Jan 2020 13:38:59 -0500
|
|
Subject: [PATCH 3/5] xutils: Change icons to being cairo surfaces
|
|
|
|
Since all icons are stored internally as cairo surfaces, we should be
|
|
returning icons as cairo surfaces from the private functions in xutils.
|
|
This simplifies the drawing codepath and makes us able to delete a bunch
|
|
of GdkPixbuf manipulation.
|
|
|
|
adapted from https://gitlab.gnome.org/GNOME/mutter/commit/af7f51b9
|
|
---
|
|
libwnck/application.c | 22 +--
|
|
libwnck/class-group.c | 17 +-
|
|
libwnck/tasklist.c | 9 +-
|
|
libwnck/window.c | 22 +--
|
|
libwnck/wnck-icon-cache-private.h | 15 +-
|
|
libwnck/wnck-icon-cache.c | 299 +++++++++++++-----------------
|
|
libwnck/xutils.c | 42 ++---
|
|
libwnck/xutils.h | 11 +-
|
|
8 files changed, 179 insertions(+), 258 deletions(-)
|
|
|
|
diff --git a/libwnck/application.c b/libwnck/application.c
|
|
index d8283cc..b441eb6 100644
|
|
--- a/libwnck/application.c
|
|
+++ b/libwnck/application.c
|
|
@@ -309,10 +309,11 @@ static void
|
|
get_icons (WnckApplication *app)
|
|
{
|
|
WnckHandle *handle;
|
|
- GdkPixbuf *icon;
|
|
- GdkPixbuf *mini_icon;
|
|
+ cairo_surface_t *icon;
|
|
+ cairo_surface_t *mini_icon;
|
|
gsize normal_size;
|
|
gsize mini_size;
|
|
+ int scaling_factor;
|
|
|
|
handle = wnck_screen_get_handle (app->priv->screen);
|
|
|
|
@@ -320,6 +321,7 @@ get_icons (WnckApplication *app)
|
|
mini_icon = NULL;
|
|
normal_size = _wnck_handle_get_default_icon_size (handle);
|
|
mini_size = _wnck_handle_get_default_mini_icon_size (handle);
|
|
+ scaling_factor = _wnck_get_window_scaling_factor ();
|
|
|
|
if (_wnck_read_icons (app->priv->screen,
|
|
app->priv->xwindow,
|
|
@@ -327,24 +329,16 @@ get_icons (WnckApplication *app)
|
|
&icon,
|
|
normal_size,
|
|
&mini_icon,
|
|
- mini_size))
|
|
+ mini_size,
|
|
+ scaling_factor))
|
|
{
|
|
app->priv->need_emit_icon_changed = TRUE;
|
|
|
|
g_clear_pointer (&app->priv->icon, cairo_surface_destroy);
|
|
g_clear_pointer (&app->priv->mini_icon, cairo_surface_destroy);
|
|
|
|
- if (icon)
|
|
- {
|
|
- app->priv->icon = gdk_cairo_surface_create_from_pixbuf (icon, 0, NULL);
|
|
- g_clear_object (&icon);
|
|
- }
|
|
-
|
|
- if (mini_icon)
|
|
- {
|
|
- app->priv->mini_icon = gdk_cairo_surface_create_from_pixbuf (mini_icon, 0, NULL);
|
|
- g_clear_object (&mini_icon);
|
|
- }
|
|
+ app->priv->icon = icon;
|
|
+ app->priv->mini_icon = mini_icon;
|
|
}
|
|
|
|
/* FIXME we should really fall back to using the icon
|
|
diff --git a/libwnck/class-group.c b/libwnck/class-group.c
|
|
index e6c45d6..7899497 100644
|
|
--- a/libwnck/class-group.c
|
|
+++ b/libwnck/class-group.c
|
|
@@ -441,23 +441,10 @@ set_icon (WnckClassGroup *class_group)
|
|
|
|
handle = wnck_screen_get_handle (class_group->priv->screen);
|
|
|
|
- GdkPixbuf *icon_pixbuf, *mini_icon_pixbuf;
|
|
-
|
|
- _wnck_get_fallback_icons (&icon_pixbuf,
|
|
+ _wnck_get_fallback_icons (&icon,
|
|
_wnck_handle_get_default_icon_size (handle),
|
|
- &mini_icon_pixbuf,
|
|
+ &mini_icon,
|
|
_wnck_handle_get_default_mini_icon_size (handle));
|
|
- if (icon_pixbuf)
|
|
- {
|
|
- icon = gdk_cairo_surface_create_from_pixbuf (icon_pixbuf, 0, NULL);
|
|
- g_clear_object (&icon_pixbuf);
|
|
- }
|
|
-
|
|
- if (mini_icon_pixbuf)
|
|
- {
|
|
- mini_icon = gdk_cairo_surface_create_from_pixbuf (mini_icon_pixbuf, 0, NULL);
|
|
- g_clear_object (&mini_icon_pixbuf);
|
|
- }
|
|
|
|
icons_reffed = TRUE;
|
|
}
|
|
diff --git a/libwnck/tasklist.c b/libwnck/tasklist.c
|
|
index 10c6cc8..3e7b789 100644
|
|
--- a/libwnck/tasklist.c
|
|
+++ b/libwnck/tasklist.c
|
|
@@ -3898,15 +3898,8 @@ wnck_task_get_icon (WnckTask *task)
|
|
|
|
if (surface == NULL)
|
|
{
|
|
- GdkPixbuf *pixbuf;
|
|
_wnck_get_fallback_icons (NULL, 0,
|
|
- &pixbuf, mini_icon_size);
|
|
-
|
|
- if (pixbuf != NULL)
|
|
- {
|
|
- surface = gdk_cairo_surface_create_from_pixbuf (pixbuf, 0, NULL);
|
|
- g_object_unref (pixbuf);
|
|
- }
|
|
+ &surface, mini_icon_size);
|
|
}
|
|
#endif
|
|
break;
|
|
diff --git a/libwnck/window.c b/libwnck/window.c
|
|
index f01b4c2..5c97675 100644
|
|
--- a/libwnck/window.c
|
|
+++ b/libwnck/window.c
|
|
@@ -2105,10 +2105,11 @@ static void
|
|
get_icons (WnckWindow *window)
|
|
{
|
|
WnckHandle *handle;
|
|
- GdkPixbuf *icon;
|
|
- GdkPixbuf *mini_icon;
|
|
+ cairo_surface_t *icon;
|
|
+ cairo_surface_t *mini_icon;
|
|
gsize normal_size;
|
|
gsize mini_size;
|
|
+ int scaling_factor;
|
|
|
|
handle = wnck_screen_get_handle (window->priv->screen);
|
|
|
|
@@ -2116,6 +2117,7 @@ get_icons (WnckWindow *window)
|
|
mini_icon = NULL;
|
|
normal_size = _wnck_handle_get_default_icon_size (handle);
|
|
mini_size = _wnck_handle_get_default_mini_icon_size (handle);
|
|
+ scaling_factor = _wnck_get_window_scaling_factor ();
|
|
|
|
if (_wnck_read_icons (window->priv->screen,
|
|
window->priv->xwindow,
|
|
@@ -2123,24 +2125,16 @@ get_icons (WnckWindow *window)
|
|
&icon,
|
|
normal_size,
|
|
&mini_icon,
|
|
- mini_size))
|
|
+ mini_size,
|
|
+ scaling_factor))
|
|
{
|
|
window->priv->need_emit_icon_changed = TRUE;
|
|
|
|
g_clear_pointer (&window->priv->icon, cairo_surface_destroy);
|
|
g_clear_pointer (&window->priv->mini_icon, cairo_surface_destroy);
|
|
|
|
- if (icon)
|
|
- {
|
|
- window->priv->icon = gdk_cairo_surface_create_from_pixbuf (icon, 0, NULL);
|
|
- g_clear_object (&icon);
|
|
- }
|
|
-
|
|
- if (mini_icon)
|
|
- {
|
|
- window->priv->mini_icon = gdk_cairo_surface_create_from_pixbuf (mini_icon, 0, NULL);
|
|
- g_clear_object (&mini_icon);
|
|
- }
|
|
+ window->priv->icon = icon;
|
|
+ window->priv->mini_icon = mini_icon;
|
|
}
|
|
|
|
g_assert ((window->priv->icon && window->priv->mini_icon) ||
|
|
diff --git a/libwnck/wnck-icon-cache-private.h b/libwnck/wnck-icon-cache-private.h
|
|
index 6a3d5ec..d3c39e2 100644
|
|
--- a/libwnck/wnck-icon-cache-private.h
|
|
+++ b/libwnck/wnck-icon-cache-private.h
|
|
@@ -38,13 +38,14 @@ void _wnck_icon_cache_set_want_fallback (WnckIconCache *icon_cache
|
|
gboolean setting);
|
|
gboolean _wnck_icon_cache_get_is_fallback (WnckIconCache *icon_cache);
|
|
|
|
-gboolean _wnck_read_icons (WnckScreen *screen,
|
|
- Window xwindow,
|
|
- WnckIconCache *icon_cache,
|
|
- GdkPixbuf **iconp,
|
|
- int ideal_size,
|
|
- GdkPixbuf **mini_iconp,
|
|
- int ideal_mini_size);
|
|
+gboolean _wnck_read_icons (WnckScreen *screen,
|
|
+ Window xwindow,
|
|
+ WnckIconCache *icon_cache,
|
|
+ cairo_surface_t **iconp,
|
|
+ int ideal_size,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int ideal_mini_size,
|
|
+ int scaling_factor);
|
|
|
|
G_END_DECLS
|
|
|
|
diff --git a/libwnck/wnck-icon-cache.c b/libwnck/wnck-icon-cache.c
|
|
index 9ff8d15..d9e67e8 100644
|
|
--- a/libwnck/wnck-icon-cache.c
|
|
+++ b/libwnck/wnck-icon-cache.c
|
|
@@ -44,8 +44,8 @@ struct _WnckIconCache
|
|
IconOrigin origin;
|
|
Pixmap prev_pixmap;
|
|
Pixmap prev_mask;
|
|
- GdkPixbuf *icon;
|
|
- GdkPixbuf *mini_icon;
|
|
+ cairo_surface_t *icon;
|
|
+ cairo_surface_t *mini_icon;
|
|
int ideal_size;
|
|
int ideal_mini_size;
|
|
guint want_fallback : 1;
|
|
@@ -139,49 +139,65 @@ find_best_size (gulong *data,
|
|
return FALSE;
|
|
}
|
|
|
|
-static void
|
|
-argbdata_to_pixdata (gulong *argb_data, int len, guchar **pixdata)
|
|
+static cairo_surface_t *
|
|
+argbdata_to_surface (gulong *argb_data,
|
|
+ int w,
|
|
+ int h,
|
|
+ int ideal_w,
|
|
+ int ideal_h,
|
|
+ int scaling_factor)
|
|
{
|
|
- guchar *p;
|
|
- int i;
|
|
+ cairo_surface_t *surface, *icon;
|
|
+ cairo_t *cr;
|
|
+ int y, x, stride;
|
|
+ uint32_t *data;
|
|
|
|
- *pixdata = g_new (guchar, len * 4);
|
|
- p = *pixdata;
|
|
+ surface = cairo_image_surface_create (CAIRO_FORMAT_ARGB32, w, h);
|
|
+ cairo_surface_set_device_scale (surface, (double)scaling_factor, (double)scaling_factor);
|
|
+ stride = cairo_image_surface_get_stride (surface) / sizeof (uint32_t);
|
|
+ data = (uint32_t *) cairo_image_surface_get_data (surface);
|
|
|
|
/* One could speed this up a lot. */
|
|
- i = 0;
|
|
- while (i < len)
|
|
+ for (y = 0; y < h; y++)
|
|
{
|
|
- guint argb;
|
|
- guint rgba;
|
|
-
|
|
- argb = argb_data[i];
|
|
- rgba = (argb << 8) | (argb >> 24);
|
|
-
|
|
- *p = rgba >> 24;
|
|
- ++p;
|
|
- *p = (rgba >> 16) & 0xff;
|
|
- ++p;
|
|
- *p = (rgba >> 8) & 0xff;
|
|
- ++p;
|
|
- *p = rgba & 0xff;
|
|
- ++p;
|
|
-
|
|
- ++i;
|
|
+ for (x = 0; x < w; x++)
|
|
+ {
|
|
+ uint32_t *p = &data[y * stride + x];
|
|
+ gulong *d = &argb_data[y * w + x];
|
|
+ *p = *d;
|
|
+ }
|
|
}
|
|
+
|
|
+ cairo_surface_mark_dirty (surface);
|
|
+
|
|
+ icon = cairo_surface_create_similar_image (surface,
|
|
+ cairo_image_surface_get_format (surface),
|
|
+ ideal_w, ideal_h);
|
|
+
|
|
+ cairo_surface_set_device_scale (icon, (double)scaling_factor, (double)scaling_factor);
|
|
+
|
|
+ cr = cairo_create (icon);
|
|
+ cairo_scale (cr, ideal_w / (double)w, ideal_h / (double)h);
|
|
+ cairo_set_source_surface (cr, surface, 0, 0);
|
|
+ cairo_paint (cr);
|
|
+
|
|
+ cairo_set_operator (cr, CAIRO_OPERATOR_IN);
|
|
+ cairo_paint (cr);
|
|
+
|
|
+ cairo_destroy (cr);
|
|
+ cairo_surface_destroy (surface);
|
|
+
|
|
+ return icon;
|
|
}
|
|
|
|
static gboolean
|
|
-read_rgb_icon (Screen *screen,
|
|
- Window xwindow,
|
|
- int ideal_size,
|
|
- int ideal_mini_size,
|
|
- int *width,
|
|
- int *height,
|
|
- guchar **pixdata,
|
|
- int *mini_width,
|
|
- int *mini_height,
|
|
- guchar **mini_pixdata)
|
|
+read_rgb_icon (Screen *screen,
|
|
+ Window xwindow,
|
|
+ int ideal_size,
|
|
+ int ideal_mini_size,
|
|
+ cairo_surface_t **iconp,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int scaling_factor)
|
|
{
|
|
Display *display;
|
|
Atom type;
|
|
@@ -219,7 +235,9 @@ read_rgb_icon (Screen *screen,
|
|
return FALSE;
|
|
}
|
|
|
|
- if (!find_best_size (data, nitems, ideal_size, &w, &h, &best))
|
|
+ if (!find_best_size (data, nitems,
|
|
+ ideal_size,
|
|
+ &w, &h, &best))
|
|
{
|
|
XFree (data);
|
|
return FALSE;
|
|
@@ -233,14 +251,8 @@ read_rgb_icon (Screen *screen,
|
|
return FALSE;
|
|
}
|
|
|
|
- *width = w;
|
|
- *height = h;
|
|
-
|
|
- *mini_width = mini_w;
|
|
- *mini_height = mini_h;
|
|
-
|
|
- argbdata_to_pixdata (best, w * h, pixdata);
|
|
- argbdata_to_pixdata (best_mini, mini_w * mini_h, mini_pixdata);
|
|
+ *iconp = argbdata_to_surface (best, w, h, ideal_size, ideal_size, scaling_factor);
|
|
+ *mini_iconp = argbdata_to_surface (best_mini, mini_w, mini_h, ideal_mini_size, ideal_mini_size, scaling_factor);
|
|
|
|
XFree (data);
|
|
|
|
@@ -248,27 +260,27 @@ read_rgb_icon (Screen *screen,
|
|
}
|
|
|
|
static gboolean
|
|
-try_pixmap_and_mask (Screen *screen,
|
|
- Pixmap src_pixmap,
|
|
- Pixmap src_mask,
|
|
- GdkPixbuf **iconp,
|
|
- int ideal_size,
|
|
- GdkPixbuf **mini_iconp,
|
|
- int ideal_mini_size)
|
|
+try_pixmap_and_mask (Screen *screen,
|
|
+ Pixmap src_pixmap,
|
|
+ Pixmap src_mask,
|
|
+ cairo_surface_t **iconp,
|
|
+ int ideal_size,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int ideal_mini_size,
|
|
+ int scaling_factor)
|
|
{
|
|
cairo_surface_t *surface, *mask_surface, *image;
|
|
GdkDisplay *gdk_display;
|
|
- GdkPixbuf *unscaled;
|
|
int width, height;
|
|
cairo_t *cr;
|
|
|
|
if (src_pixmap == None)
|
|
return FALSE;
|
|
|
|
- surface = _wnck_cairo_surface_get_from_pixmap (screen, src_pixmap);
|
|
+ surface = _wnck_cairo_surface_get_from_pixmap (screen, src_pixmap, scaling_factor);
|
|
|
|
if (surface && src_mask != None)
|
|
- mask_surface = _wnck_cairo_surface_get_from_pixmap (screen, src_mask);
|
|
+ mask_surface = _wnck_cairo_surface_get_from_pixmap (screen, src_mask, scaling_factor);
|
|
else
|
|
mask_surface = NULL;
|
|
|
|
@@ -324,26 +336,41 @@ try_pixmap_and_mask (Screen *screen,
|
|
return FALSE;
|
|
}
|
|
|
|
- unscaled = gdk_pixbuf_get_from_surface (image,
|
|
- 0, 0,
|
|
- width, height);
|
|
+ if (image)
|
|
+ {
|
|
+ int image_w, image_h;
|
|
|
|
- cairo_surface_destroy (image);
|
|
+ image_w = cairo_image_surface_get_width (image);
|
|
+ image_h = cairo_image_surface_get_height (image);
|
|
+
|
|
+ *iconp = cairo_surface_create_similar (image,
|
|
+ cairo_surface_get_content (image),
|
|
+ ideal_size,
|
|
+ ideal_size);
|
|
+
|
|
+ cairo_surface_set_device_scale (*iconp, (double)scaling_factor, (double)scaling_factor);
|
|
+
|
|
+ cr = cairo_create (*iconp);
|
|
+ cairo_scale (cr, ideal_size / (double)image_w, ideal_size / (double)image_h);
|
|
+ cairo_set_source_surface (cr, image, 0, 0);
|
|
+ cairo_paint (cr);
|
|
+ cairo_destroy (cr);
|
|
+
|
|
+ *mini_iconp = cairo_surface_create_similar (image,
|
|
+ cairo_surface_get_content (image),
|
|
+ ideal_mini_size,
|
|
+ ideal_mini_size);
|
|
+
|
|
+ cairo_surface_set_device_scale (*mini_iconp, (double)scaling_factor, (double)scaling_factor);
|
|
+
|
|
+ cr = cairo_create (*mini_iconp);
|
|
+ cairo_scale (cr, ideal_mini_size / (double)image_w, ideal_mini_size / (double)image_h);
|
|
+ cairo_set_source_surface (cr, image, 0, 0);
|
|
+ cairo_paint (cr);
|
|
+ cairo_destroy (cr);
|
|
+
|
|
+ cairo_surface_destroy (image);
|
|
|
|
- if (unscaled)
|
|
- {
|
|
- *iconp =
|
|
- gdk_pixbuf_scale_simple (unscaled,
|
|
- ideal_size,
|
|
- ideal_size,
|
|
- GDK_INTERP_BILINEAR);
|
|
- *mini_iconp =
|
|
- gdk_pixbuf_scale_simple (unscaled,
|
|
- ideal_mini_size,
|
|
- ideal_mini_size,
|
|
- GDK_INTERP_BILINEAR);
|
|
-
|
|
- g_object_unref (G_OBJECT (unscaled));
|
|
return TRUE;
|
|
}
|
|
else
|
|
@@ -354,13 +381,8 @@ static void
|
|
clear_icon_cache (WnckIconCache *icon_cache,
|
|
gboolean dirty_all)
|
|
{
|
|
- if (icon_cache->icon)
|
|
- g_object_unref (G_OBJECT (icon_cache->icon));
|
|
- icon_cache->icon = NULL;
|
|
-
|
|
- if (icon_cache->mini_icon)
|
|
- g_object_unref (G_OBJECT (icon_cache->mini_icon));
|
|
- icon_cache->mini_icon = NULL;
|
|
+ g_clear_pointer (&icon_cache->icon, cairo_surface_destroy);
|
|
+ g_clear_pointer (&icon_cache->mini_icon, cairo_surface_destroy);
|
|
|
|
icon_cache->origin = USING_NO_ICON;
|
|
|
|
@@ -372,89 +394,26 @@ clear_icon_cache (WnckIconCache *icon_cache,
|
|
}
|
|
|
|
static void
|
|
-replace_cache (WnckIconCache *icon_cache,
|
|
- IconOrigin origin,
|
|
- GdkPixbuf *new_icon,
|
|
- GdkPixbuf *new_mini_icon)
|
|
+replace_cache (WnckIconCache *icon_cache,
|
|
+ IconOrigin origin,
|
|
+ cairo_surface_t *new_icon,
|
|
+ cairo_surface_t *new_mini_icon)
|
|
{
|
|
clear_icon_cache (icon_cache, FALSE);
|
|
|
|
icon_cache->origin = origin;
|
|
|
|
if (new_icon)
|
|
- g_object_ref (G_OBJECT (new_icon));
|
|
+ cairo_surface_reference (new_icon);
|
|
|
|
icon_cache->icon = new_icon;
|
|
|
|
if (new_mini_icon)
|
|
- g_object_ref (G_OBJECT (new_mini_icon));
|
|
+ cairo_surface_reference (new_mini_icon);
|
|
|
|
icon_cache->mini_icon = new_mini_icon;
|
|
}
|
|
|
|
-static void
|
|
-free_pixels (guchar *pixels,
|
|
- gpointer data)
|
|
-{
|
|
- g_free (pixels);
|
|
-}
|
|
-
|
|
-static GdkPixbuf*
|
|
-scaled_from_pixdata (guchar *pixdata,
|
|
- int w,
|
|
- int h,
|
|
- int new_w,
|
|
- int new_h)
|
|
-{
|
|
- GdkPixbuf *src;
|
|
- GdkPixbuf *dest;
|
|
-
|
|
- src = gdk_pixbuf_new_from_data (pixdata,
|
|
- GDK_COLORSPACE_RGB,
|
|
- TRUE,
|
|
- 8,
|
|
- w, h, w * 4,
|
|
- free_pixels,
|
|
- NULL);
|
|
-
|
|
- if (src == NULL)
|
|
- return NULL;
|
|
-
|
|
- if (w != h)
|
|
- {
|
|
- GdkPixbuf *tmp;
|
|
- int size;
|
|
-
|
|
- size = MAX (w, h);
|
|
-
|
|
- tmp = gdk_pixbuf_new (GDK_COLORSPACE_RGB, TRUE, 8, size, size);
|
|
-
|
|
- if (tmp != NULL)
|
|
- {
|
|
- gdk_pixbuf_fill (tmp, 0);
|
|
- gdk_pixbuf_copy_area (src, 0, 0, w, h,
|
|
- tmp,
|
|
- (size - w) / 2, (size - h) / 2);
|
|
-
|
|
- g_object_unref (src);
|
|
- src = tmp;
|
|
- }
|
|
- }
|
|
-
|
|
- if (w != new_w || h != new_h)
|
|
- {
|
|
- dest = gdk_pixbuf_scale_simple (src, new_w, new_h, GDK_INTERP_BILINEAR);
|
|
-
|
|
- g_object_unref (G_OBJECT (src));
|
|
- }
|
|
- else
|
|
- {
|
|
- dest = src;
|
|
- }
|
|
-
|
|
- return dest;
|
|
-}
|
|
-
|
|
WnckIconCache*
|
|
_wnck_icon_cache_new (void)
|
|
{
|
|
@@ -528,22 +487,17 @@ _wnck_icon_cache_get_is_fallback (WnckIconCache *icon_cache)
|
|
}
|
|
|
|
gboolean
|
|
-_wnck_read_icons (WnckScreen *screen,
|
|
- Window xwindow,
|
|
- WnckIconCache *icon_cache,
|
|
- GdkPixbuf **iconp,
|
|
- int ideal_size,
|
|
- GdkPixbuf **mini_iconp,
|
|
- int ideal_mini_size)
|
|
+_wnck_read_icons (WnckScreen *screen,
|
|
+ Window xwindow,
|
|
+ WnckIconCache *icon_cache,
|
|
+ cairo_surface_t **iconp,
|
|
+ int ideal_size,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int ideal_mini_size,
|
|
+ int scaling_factor)
|
|
{
|
|
Screen *xscreen;
|
|
Display *display;
|
|
- guchar *pixdata;
|
|
- int w, h;
|
|
- guchar *mini_pixdata;
|
|
- int mini_w, mini_h;
|
|
- Pixmap pixmap;
|
|
- Pixmap mask;
|
|
XWMHints *hints;
|
|
|
|
/* Return value is whether the icon changed */
|
|
@@ -556,6 +510,9 @@ _wnck_read_icons (WnckScreen *screen,
|
|
*iconp = NULL;
|
|
*mini_iconp = NULL;
|
|
|
|
+ ideal_size *= scaling_factor;
|
|
+ ideal_mini_size *= scaling_factor;
|
|
+
|
|
if (ideal_size != icon_cache->ideal_size ||
|
|
ideal_mini_size != icon_cache->ideal_mini_size)
|
|
clear_icon_cache (icon_cache, TRUE);
|
|
@@ -566,8 +523,6 @@ _wnck_read_icons (WnckScreen *screen,
|
|
if (!_wnck_icon_cache_get_icon_invalidated (icon_cache))
|
|
return FALSE; /* we have no new info to use */
|
|
|
|
- pixdata = NULL;
|
|
-
|
|
/* Our algorithm here assumes that we can't have for example origin
|
|
* < USING_NET_WM_ICON and icon_cache->net_wm_icon_dirty == FALSE
|
|
* unless we have tried to read NET_WM_ICON.
|
|
@@ -579,21 +534,15 @@ _wnck_read_icons (WnckScreen *screen,
|
|
|
|
if (icon_cache->origin <= USING_NET_WM_ICON &&
|
|
icon_cache->net_wm_icon_dirty)
|
|
-
|
|
{
|
|
icon_cache->net_wm_icon_dirty = FALSE;
|
|
|
|
if (read_rgb_icon (xscreen, xwindow,
|
|
ideal_size,
|
|
ideal_mini_size,
|
|
- &w, &h, &pixdata,
|
|
- &mini_w, &mini_h, &mini_pixdata))
|
|
+ iconp, mini_iconp,
|
|
+ scaling_factor))
|
|
{
|
|
- *iconp = scaled_from_pixdata (pixdata, w, h, ideal_size, ideal_size);
|
|
-
|
|
- *mini_iconp = scaled_from_pixdata (mini_pixdata, mini_w, mini_h,
|
|
- ideal_mini_size, ideal_mini_size);
|
|
-
|
|
replace_cache (icon_cache, USING_NET_WM_ICON,
|
|
*iconp, *mini_iconp);
|
|
|
|
@@ -604,6 +553,9 @@ _wnck_read_icons (WnckScreen *screen,
|
|
if (icon_cache->origin <= USING_WM_HINTS &&
|
|
icon_cache->wm_hints_dirty)
|
|
{
|
|
+ Pixmap pixmap;
|
|
+ Pixmap mask;
|
|
+
|
|
icon_cache->wm_hints_dirty = FALSE;
|
|
|
|
_wnck_error_trap_push (display);
|
|
@@ -632,7 +584,8 @@ _wnck_read_icons (WnckScreen *screen,
|
|
{
|
|
if (try_pixmap_and_mask (xscreen, pixmap, mask,
|
|
iconp, ideal_size,
|
|
- mini_iconp, ideal_mini_size))
|
|
+ mini_iconp, ideal_mini_size,
|
|
+ scaling_factor))
|
|
{
|
|
icon_cache->prev_pixmap = pixmap;
|
|
icon_cache->prev_mask = mask;
|
|
diff --git a/libwnck/xutils.c b/libwnck/xutils.c
|
|
index 60ae7b2..476f027 100644
|
|
--- a/libwnck/xutils.c
|
|
+++ b/libwnck/xutils.c
|
|
@@ -1389,7 +1389,8 @@ _wnck_select_input (Screen *screen,
|
|
|
|
cairo_surface_t *
|
|
_wnck_cairo_surface_get_from_pixmap (Screen *screen,
|
|
- Pixmap xpixmap)
|
|
+ Pixmap xpixmap,
|
|
+ int scaling_factor)
|
|
{
|
|
cairo_surface_t *surface;
|
|
Display *display;
|
|
@@ -1407,6 +1408,9 @@ _wnck_cairo_surface_get_from_pixmap (Screen *screen,
|
|
&x_ret, &y_ret, &w_ret, &h_ret, &bw_ret, &depth_ret))
|
|
goto TRAP_POP;
|
|
|
|
+ w_ret *= scaling_factor;
|
|
+ h_ret *= scaling_factor;
|
|
+
|
|
if (depth_ret == 1)
|
|
{
|
|
surface = cairo_xlib_surface_create_for_bitmap (display,
|
|
@@ -1463,7 +1467,7 @@ _wnck_gdk_pixbuf_get_from_pixmap (Screen *screen,
|
|
cairo_surface_t *surface;
|
|
GdkPixbuf *retval;
|
|
|
|
- surface = _wnck_cairo_surface_get_from_pixmap (screen, xpixmap);
|
|
+ surface = _wnck_cairo_surface_get_from_pixmap (screen, xpixmap, 1);
|
|
|
|
if (surface == NULL)
|
|
return NULL;
|
|
@@ -1478,36 +1482,30 @@ _wnck_gdk_pixbuf_get_from_pixmap (Screen *screen,
|
|
return retval;
|
|
}
|
|
|
|
-static GdkPixbuf*
|
|
+static cairo_surface_t*
|
|
default_icon_at_size (int size)
|
|
{
|
|
- GdkPixbuf *base;
|
|
+ GdkPixbuf *pixbuf;
|
|
+ cairo_surface_t *surface;
|
|
|
|
- base = gdk_pixbuf_new_from_resource ("/org/gnome/libwnck/default_icon.png", NULL);
|
|
+ pixbuf = gdk_pixbuf_new_from_resource_at_scale ("/org/gnome/libwnck/default_icon.png",
|
|
+ size, size,
|
|
+ TRUE, NULL);
|
|
|
|
- g_assert (base);
|
|
+ g_assert (pixbuf);
|
|
|
|
- if (gdk_pixbuf_get_width (base) == size &&
|
|
- gdk_pixbuf_get_height (base) == size)
|
|
- {
|
|
- return base;
|
|
- }
|
|
- else
|
|
- {
|
|
- GdkPixbuf *scaled;
|
|
+ surface = gdk_cairo_surface_create_from_pixbuf (pixbuf, 0, NULL);
|
|
|
|
- scaled = gdk_pixbuf_scale_simple (base, size, size, GDK_INTERP_BILINEAR);
|
|
- g_object_unref (G_OBJECT (base));
|
|
+ g_clear_object (&pixbuf);
|
|
|
|
- return scaled;
|
|
- }
|
|
+ return surface;
|
|
}
|
|
|
|
void
|
|
-_wnck_get_fallback_icons (GdkPixbuf **iconp,
|
|
- int ideal_size,
|
|
- GdkPixbuf **mini_iconp,
|
|
- int ideal_mini_size)
|
|
+_wnck_get_fallback_icons (cairo_surface_t **iconp,
|
|
+ int ideal_size,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int ideal_mini_size)
|
|
{
|
|
if (iconp)
|
|
*iconp = default_icon_at_size (ideal_size);
|
|
diff --git a/libwnck/xutils.h b/libwnck/xutils.h
|
|
index 4e5c620..51e8e55 100644
|
|
--- a/libwnck/xutils.h
|
|
+++ b/libwnck/xutils.h
|
|
@@ -156,10 +156,10 @@ void _wnck_keyboard_size (WnckScreen *screen,
|
|
void _wnck_toggle_showing_desktop (Screen *screen,
|
|
gboolean show);
|
|
|
|
-void _wnck_get_fallback_icons (GdkPixbuf **iconp,
|
|
- int ideal_size,
|
|
- GdkPixbuf **mini_iconp,
|
|
- int ideal_mini_size);
|
|
+void _wnck_get_fallback_icons (cairo_surface_t **iconp,
|
|
+ int ideal_size,
|
|
+ cairo_surface_t **mini_iconp,
|
|
+ int ideal_mini_size);
|
|
|
|
void _wnck_get_window_geometry (Screen *screen,
|
|
Window xwindow,
|
|
@@ -192,7 +192,8 @@ void _wnck_set_desktop_layout (Screen *xscreen,
|
|
int columns);
|
|
|
|
cairo_surface_t *_wnck_cairo_surface_get_from_pixmap (Screen *screen,
|
|
- Pixmap xpixmap);
|
|
+ Pixmap xpixmap,
|
|
+ int scaling_factor);
|
|
|
|
GdkPixbuf* _wnck_gdk_pixbuf_get_from_pixmap (Screen *screen,
|
|
Pixmap xpixmap);
|
|
--
|
|
2.37.2
|
|
|