iconhelper: transition to GdkPaintable

This is in preparation for accepting the image type paintable.

It's a bit incovenient because we need more code to track width/height
ourselves (as the paintable no longer does it for us), but it's not too
hard.

GtkIconHelper does not track invalidations on the paintable.
This commit is contained in:
Benjamin Otte 2018-02-16 09:09:35 +01:00
parent a4e16ce3cc
commit c59948169d
4 changed files with 86 additions and 78 deletions

View File

@ -38,7 +38,7 @@
void
gtk_icon_helper_invalidate (GtkIconHelper *self)
{
g_clear_object (&self->texture);
g_clear_object (&self->paintable);
self->texture_scale = 1;
self->texture_is_symbolic = FALSE;
@ -57,7 +57,7 @@ gtk_icon_helper_invalidate_for_change (GtkIconHelper *self,
!self->texture_is_symbolic)))
{
/* Avoid the queue_resize in gtk_icon_helper_invalidate */
g_clear_object (&self->texture);
g_clear_object (&self->paintable);
self->texture_scale = 1;
self->texture_is_symbolic = FALSE;
@ -86,7 +86,7 @@ gtk_icon_helper_take_definition (GtkIconHelper *self,
void
_gtk_icon_helper_clear (GtkIconHelper *self)
{
g_clear_object (&self->texture);
g_clear_object (&self->paintable);
self->texture_scale = 1;
self->texture_is_symbolic = FALSE;
@ -210,10 +210,10 @@ get_surface_size (cairo_surface_t *surface,
cairo_destroy (cr);
}
static GdkTexture *
ensure_texture_from_surface (GtkIconHelper *self,
cairo_surface_t *orig_surface,
int *scale_out)
static GdkPaintable *
ensure_paintable_from_surface (GtkIconHelper *self,
cairo_surface_t *orig_surface,
int *scale_out)
{
cairo_surface_t *map;
int width, height, scale;
@ -255,26 +255,26 @@ ensure_texture_from_surface (GtkIconHelper *self,
cairo_surface_unmap_image (orig_surface, map);
return texture;
return GDK_PAINTABLE (texture);
}
static GdkTexture *
ensure_texture_from_texture (GtkIconHelper *self,
GdkTexture *texture,
int *scale)
static GdkPaintable *
ensure_paintable_from_texture (GtkIconHelper *self,
GdkTexture *texture,
int *scale)
{
*scale = 1;
return g_object_ref (texture);
return g_object_ref (GDK_PAINTABLE (texture));
}
static GdkTexture *
ensure_texture_for_gicon (GtkIconHelper *self,
GtkCssStyle *style,
GtkTextDirection dir,
gint scale,
GIcon *gicon,
gboolean *symbolic)
static GdkPaintable *
ensure_paintable_for_gicon (GtkIconHelper *self,
GtkCssStyle *style,
GtkTextDirection dir,
gint scale,
GIcon *gicon,
gboolean *symbolic)
{
GtkIconTheme *icon_theme;
gint width, height;
@ -301,15 +301,15 @@ ensure_texture_for_gicon (GtkIconHelper *self,
*symbolic = gtk_icon_info_is_symbolic (info);
texture = gtk_icon_info_load_texture (info);
return texture;
return GDK_PAINTABLE (texture);
}
static GdkTexture *
gtk_icon_helper_load_texture (GtkIconHelper *self,
int *out_scale,
gboolean *out_symbolic)
static GdkPaintable *
gtk_icon_helper_load_paintable (GtkIconHelper *self,
int *out_scale,
gboolean *out_symbolic)
{
GdkTexture *texture;
GdkPaintable *paintable;
GIcon *gicon;
int scale;
gboolean symbolic;
@ -317,12 +317,12 @@ gtk_icon_helper_load_texture (GtkIconHelper *self,
switch (gtk_image_definition_get_storage_type (self->def))
{
case GTK_IMAGE_SURFACE:
texture = ensure_texture_from_surface (self, gtk_image_definition_get_surface (self->def), &scale);
paintable = ensure_paintable_from_surface (self, gtk_image_definition_get_surface (self->def), &scale);
symbolic = FALSE;
break;
case GTK_IMAGE_TEXTURE:
texture = ensure_texture_from_texture (self, gtk_image_definition_get_texture (self->def), &scale);
paintable = ensure_paintable_from_texture (self, gtk_image_definition_get_texture (self->def), &scale);
symbolic = FALSE;
break;
@ -332,28 +332,28 @@ gtk_icon_helper_load_texture (GtkIconHelper *self,
gicon = g_themed_icon_new_with_default_fallbacks (gtk_image_definition_get_icon_name (self->def));
else
gicon = g_themed_icon_new (gtk_image_definition_get_icon_name (self->def));
texture = ensure_texture_for_gicon (self,
gtk_css_node_get_style (self->node),
gtk_widget_get_direction (self->owner),
scale,
gicon,
&symbolic);
paintable = ensure_paintable_for_gicon (self,
gtk_css_node_get_style (self->node),
gtk_widget_get_direction (self->owner),
scale,
gicon,
&symbolic);
g_object_unref (gicon);
break;
case GTK_IMAGE_GICON:
scale = gtk_widget_get_scale_factor (self->owner);
texture = ensure_texture_for_gicon (self,
gtk_css_node_get_style (self->node),
gtk_widget_get_direction (self->owner),
scale,
gtk_image_definition_get_gicon (self->def),
&symbolic);
paintable = ensure_paintable_for_gicon (self,
gtk_css_node_get_style (self->node),
gtk_widget_get_direction (self->owner),
scale,
gtk_image_definition_get_gicon (self->def),
&symbolic);
break;
case GTK_IMAGE_EMPTY:
default:
texture = NULL;
paintable = NULL;
scale = 1;
symbolic = FALSE;
break;
@ -362,20 +362,20 @@ gtk_icon_helper_load_texture (GtkIconHelper *self,
*out_scale = scale;
*out_symbolic = symbolic;
return texture;
return paintable;
}
static void
gtk_icon_helper_ensure_texture (GtkIconHelper *self)
gtk_icon_helper_ensure_paintable (GtkIconHelper *self)
{
gboolean symbolic;
if (self->texture)
if (self->paintable)
return;
self->texture = gtk_icon_helper_load_texture (self,
&self->texture_scale,
&symbolic);
self->paintable = gtk_icon_helper_load_paintable (self,
&self->texture_scale,
&symbolic);
self->texture_is_symbolic = symbolic;
}
@ -421,12 +421,21 @@ _gtk_icon_helper_get_size (GtkIconHelper *self,
/* Otherwise we load the surface to guarantee we get a size */
if (width == 0)
{
gtk_icon_helper_ensure_texture (self);
gtk_icon_helper_ensure_paintable (self);
if (self->texture != NULL)
if (self->paintable != NULL)
{
width = (gdk_texture_get_width (self->texture) + self->texture_scale - 1) / self->texture_scale;
height = (gdk_texture_get_height (self->texture) + self->texture_scale - 1) / self->texture_scale;
width = gdk_paintable_get_intrinsic_width (self->paintable);
height = gdk_paintable_get_intrinsic_height (self->paintable);
if (width == 0 || height == 0)
{
ensure_icon_size (self, &width, &height);
}
else
{
width = (width + self->texture_scale - 1) / self->texture_scale;
height = (height + self->texture_scale - 1) / self->texture_scale;
}
}
else
{
@ -557,18 +566,21 @@ gtk_icon_helper_snapshot (GtkIconHelper *self,
GtkSnapshot *snapshot)
{
GtkCssStyle *style;
gint width, height;
style = gtk_css_node_get_style (self->node);
gtk_icon_helper_ensure_texture (self);
if (self->texture == NULL)
gtk_icon_helper_ensure_paintable (self);
if (self->paintable == NULL)
return;
gtk_css_style_snapshot_icon_texture (style,
snapshot,
self->texture,
self->texture_scale,
self->texture_is_symbolic);
_gtk_icon_helper_get_size (self, &width, &height);
gtk_css_style_snapshot_icon_paintable (style,
snapshot,
self->paintable,
width, height,
self->texture_is_symbolic);
}
gboolean

View File

@ -44,7 +44,7 @@ struct _GtkIconHelper
GtkWidget *owner;
GtkCssNode *node;
GdkTexture *texture;
GdkPaintable *paintable;
int texture_scale;
};

View File

@ -265,28 +265,26 @@ gtk_css_style_render_icon_get_extents (GtkCssStyle *style,
}
void
gtk_css_style_snapshot_icon_texture (GtkCssStyle *style,
GtkSnapshot *snapshot,
GdkTexture *texture,
double texture_scale,
gboolean recolor)
gtk_css_style_snapshot_icon_paintable (GtkCssStyle *style,
GtkSnapshot *snapshot,
GdkPaintable *paintable,
double width,
double height,
gboolean recolor)
{
const GtkCssValue *shadows_value, *transform_value, *filter_value;
graphene_matrix_t transform_matrix;
graphene_rect_t bounds;
double width, height;
gboolean has_shadow;
g_return_if_fail (GTK_IS_CSS_STYLE (style));
g_return_if_fail (snapshot != NULL);
g_return_if_fail (GDK_IS_TEXTURE (texture));
g_return_if_fail (texture_scale > 0);
g_return_if_fail (GDK_IS_PAINTABLE (paintable));
g_return_if_fail (width > 0);
g_return_if_fail (height > 0);
shadows_value = gtk_css_style_get_value (style, GTK_CSS_PROPERTY_ICON_SHADOW);
transform_value = gtk_css_style_get_value (style, GTK_CSS_PROPERTY_ICON_TRANSFORM);
filter_value = gtk_css_style_get_value (style, GTK_CSS_PROPERTY_ICON_FILTER);
width = gdk_texture_get_width (texture) / texture_scale;
height = gdk_texture_get_height (texture) / texture_scale;
if (!gtk_css_transform_value_get_matrix (transform_value, &transform_matrix))
return;
@ -316,8 +314,7 @@ gtk_css_style_snapshot_icon_texture (GtkCssStyle *style,
if (graphene_matrix_is_identity (&transform_matrix))
{
graphene_rect_init (&bounds, 0, 0, width, height);
gtk_snapshot_append_texture (snapshot, texture, &bounds, "Icon");
gdk_paintable_snapshot (paintable, snapshot, width, height);
}
else
{
@ -328,12 +325,10 @@ gtk_css_style_snapshot_icon_texture (GtkCssStyle *style,
graphene_matrix_multiply (&transform_matrix, &m1, &m3);
graphene_matrix_init_translate (&m2, &GRAPHENE_POINT3D_INIT (- width / 2.0, - height / 2.0, 0));
graphene_matrix_multiply (&m2, &m3, &m1);
graphene_matrix_scale (&m1, 1.0 / texture_scale, 1.0 / texture_scale, 1);
gtk_snapshot_push_transform (snapshot, &m1, "Icon Transform");
graphene_rect_init (&bounds, 0, 0, gdk_texture_get_width (texture), gdk_texture_get_height (texture));
gtk_snapshot_append_texture (snapshot, texture, &bounds, "Icon");
gdk_paintable_snapshot (paintable, snapshot, width, height);
gtk_snapshot_pop (snapshot);
}

View File

@ -47,10 +47,11 @@ void gtk_css_style_render_icon_surface (GtkCssStyle *style,
cairo_surface_t *surface,
double x,
double y);
void gtk_css_style_snapshot_icon_texture (GtkCssStyle *style,
void gtk_css_style_snapshot_icon_paintable (GtkCssStyle *style,
GtkSnapshot *snapshot,
GdkTexture *texture,
double texture_scale,
GdkPaintable *paintable,
double width,
double height,
gboolean recolor);
void gtk_css_style_render_icon_get_extents (GtkCssStyle *style,