[gtksourceview/wip/chergert/gsv-gtk4: 63/84] pixbufhelper: port to GdkPaintable



commit 8bdad507d09b80831b8e4915f875090b930aa14f
Author: Christian Hergert <chergert redhat com>
Date:   Wed Jan 15 14:25:51 2020 -0800

    pixbufhelper: port to GdkPaintable
    
    We want to get a GdkPaintable from the IconCache as well to provide to
    various GTK API so that we don't take on costly copies or format
    conversions when rendering with the GPU.

 gtksourceview/gtksourcepixbufhelper-private.h |  2 +-
 gtksourceview/gtksourcepixbufhelper.c         | 96 ++++++++++-----------------
 2 files changed, 36 insertions(+), 62 deletions(-)
---
diff --git a/gtksourceview/gtksourcepixbufhelper-private.h b/gtksourceview/gtksourcepixbufhelper-private.h
index 58904256..d7d22b80 100644
--- a/gtksourceview/gtksourcepixbufhelper-private.h
+++ b/gtksourceview/gtksourcepixbufhelper-private.h
@@ -46,7 +46,7 @@ void                   gtk_source_pixbuf_helper_set_gicon     (GtkSourcePixbufHe
 G_GNUC_INTERNAL
 GIcon                 *gtk_source_pixbuf_helper_get_gicon     (GtkSourcePixbufHelper *helper);
 G_GNUC_INTERNAL
-GdkPixbuf             *gtk_source_pixbuf_helper_render        (GtkSourcePixbufHelper *helper,
+GdkPaintable          *gtk_source_pixbuf_helper_render        (GtkSourcePixbufHelper *helper,
                                                                GtkWidget             *widget,
                                                                gint                   size);
 
diff --git a/gtksourceview/gtksourcepixbufhelper.c b/gtksourceview/gtksourcepixbufhelper.c
index 6af12974..3ecba51a 100644
--- a/gtksourceview/gtksourcepixbufhelper.c
+++ b/gtksourceview/gtksourcepixbufhelper.c
@@ -31,7 +31,7 @@ typedef enum _IconType
 
 struct _GtkSourcePixbufHelper
 {
-       GdkPixbuf *cached_pixbuf;
+       GdkPaintable *cached_paintable;
        IconType type;
 
        GdkPixbuf *pixbuf;
@@ -48,40 +48,20 @@ gtk_source_pixbuf_helper_new (void)
 void
 gtk_source_pixbuf_helper_free (GtkSourcePixbufHelper *helper)
 {
-       if (helper->pixbuf)
-       {
-               g_object_unref (helper->pixbuf);
-       }
-
-       if (helper->cached_pixbuf)
-       {
-               g_object_unref (helper->cached_pixbuf);
-       }
-
-       if (helper->gicon)
-       {
-               g_object_unref (helper->gicon);
-       }
-
-       g_free (helper->icon_name);
+       g_clear_object (&helper->pixbuf);
+       g_clear_object (&helper->cached_paintable);
+       g_clear_object (&helper->gicon);
+       g_clear_pointer (&helper->icon_name, g_free);
 
        g_slice_free (GtkSourcePixbufHelper, helper);
 }
 
 static void
 set_cache (GtkSourcePixbufHelper *helper,
-           GdkPixbuf             *pixbuf)
+          GdkPaintable          *paintable)
 {
-       if (helper->cached_pixbuf)
-       {
-               g_object_unref (helper->cached_pixbuf);
-               helper->cached_pixbuf = NULL;
-       }
-
-       if (pixbuf)
-       {
-               helper->cached_pixbuf = pixbuf;
-       }
+       g_clear_object (&helper->cached_paintable);
+       helper->cached_paintable = paintable;
 }
 
 static void
@@ -169,26 +149,10 @@ from_pixbuf (GtkSourcePixbufHelper *helper,
              GtkWidget             *widget,
              gint                   size)
 {
-       if (helper->pixbuf == NULL)
+       if (helper->pixbuf != NULL)
        {
-               return;
-       }
-
-       if (gdk_pixbuf_get_width (helper->pixbuf) <= size)
-       {
-               if (!helper->cached_pixbuf)
-               {
-                       set_cache (helper, gdk_pixbuf_copy (helper->pixbuf));
-               }
-
-               return;
+               set_cache (helper, GDK_PAINTABLE (gdk_texture_new_for_pixbuf (helper->pixbuf)));
        }
-
-       /* Make smaller */
-       set_cache (helper, gdk_pixbuf_scale_simple (helper->pixbuf,
-                                                   size,
-                                                   size,
-                                                   GDK_INTERP_BILINEAR));
 }
 
 static void
@@ -196,13 +160,18 @@ from_gicon (GtkSourcePixbufHelper *helper,
             GtkWidget             *widget,
             gint                   size)
 {
-       GdkScreen *screen;
+       GdkDisplay *display;
        GtkIconTheme *icon_theme;
        GtkIconInfo *info;
        GtkIconLookupFlags flags;
 
-       screen = gtk_widget_get_screen (widget);
-       icon_theme = gtk_icon_theme_get_for_screen (screen);
+       if (helper->gicon == NULL)
+       {
+               return;
+       }
+
+       display = gtk_widget_get_display (widget);
+       icon_theme = gtk_icon_theme_get_for_display (display);
 
        flags = GTK_ICON_LOOKUP_USE_BUILTIN;
 
@@ -213,6 +182,7 @@ from_gicon (GtkSourcePixbufHelper *helper,
 
        if (info)
        {
+
                set_cache (helper, gtk_icon_info_load_icon (info, NULL));
        }
 }
@@ -222,14 +192,19 @@ from_name (GtkSourcePixbufHelper *helper,
            GtkWidget             *widget,
            gint                   size)
 {
-       GdkScreen *screen;
+       GdkDisplay *display;
        GtkIconTheme *icon_theme;
        GtkIconInfo *info;
        GtkIconLookupFlags flags;
        gint scale;
 
-       screen = gtk_widget_get_screen (widget);
-       icon_theme = gtk_icon_theme_get_for_screen (screen);
+       if (helper->icon_name == NULL)
+       {
+               return;
+       }
+
+       display = gtk_widget_get_display (widget);
+       icon_theme = gtk_icon_theme_get_for_display (display);
 
        flags = GTK_ICON_LOOKUP_USE_BUILTIN;
         scale = gtk_widget_get_scale_factor (widget);
@@ -242,33 +217,32 @@ from_name (GtkSourcePixbufHelper *helper,
 
        if (info)
        {
-               GdkPixbuf *pixbuf;
+               GdkPaintable *paintable;
 
                if (gtk_icon_info_is_symbolic (info))
                {
                        GtkStyleContext *context;
 
                        context = gtk_widget_get_style_context (widget);
-                       pixbuf = gtk_icon_info_load_symbolic_for_context (info, context, NULL, NULL);
+                       paintable = gtk_icon_info_load_symbolic_for_context (info, context, NULL, NULL);
                }
                else
                {
-                       pixbuf = gtk_icon_info_load_icon (info, NULL);
+                       paintable = gtk_icon_info_load_icon (info, NULL);
                }
 
-               set_cache (helper, pixbuf);
+               set_cache (helper, paintable);
        }
 }
 
-GdkPixbuf *
+GdkPaintable *
 gtk_source_pixbuf_helper_render (GtkSourcePixbufHelper *helper,
                                  GtkWidget             *widget,
                                  gint                   size)
 {
-       if (helper->cached_pixbuf &&
-           gdk_pixbuf_get_width (helper->cached_pixbuf) == size)
+       if (helper->cached_paintable != NULL)
        {
-               return helper->cached_pixbuf;
+               return helper->cached_paintable;
        }
 
        switch (helper->type)
@@ -286,6 +260,6 @@ gtk_source_pixbuf_helper_render (GtkSourcePixbufHelper *helper,
                        g_assert_not_reached ();
        }
 
-       return helper->cached_pixbuf;
+       return helper->cached_paintable;
 }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]