]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkcellrendererpixbuf.c
label: Fix memleak
[~andy/gtk] / gtk / gtkcellrendererpixbuf.c
index f50b3aeee9d6322191f368ccc6a28332f19f1c58..90fc9fece085141ccd7fb28a987963d1fd066844 100644 (file)
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public
- * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * License along with this library. If not, see <http://www.gnu.org/licenses/>.
  */
 
 #include "config.h"
 #include <stdlib.h>
 #include "gtkcellrendererpixbuf.h"
 #include "gtkiconfactory.h"
+#include "gtkiconhelperprivate.h"
 #include "gtkicontheme.h"
 #include "gtkintl.h"
 #include "gtkprivate.h"
+#include "a11y/gtkimagecellaccessible.h"
 
 
 /**
@@ -54,9 +54,6 @@ static void gtk_cell_renderer_pixbuf_set_property  (GObject                    *
                                                    guint                       param_id,
                                                    const GValue               *value,
                                                    GParamSpec                 *pspec);
-static void gtk_cell_renderer_pixbuf_finalize   (GObject                    *object);
-static void gtk_cell_renderer_pixbuf_create_stock_pixbuf (GtkCellRendererPixbuf *cellpixbuf,
-                                                         GtkWidget             *widget);
 static void gtk_cell_renderer_pixbuf_get_size   (GtkCellRenderer            *cell,
                                                 GtkWidget                  *widget,
                                                 const GdkRectangle         *rectangle,
@@ -88,19 +85,15 @@ enum {
 
 struct _GtkCellRendererPixbufPrivate
 {
-  GtkIconSize stock_size;
+  GtkIconHelper *icon_helper;
+  GtkIconSize    icon_size;
 
-  GdkPixbuf *pixbuf;
   GdkPixbuf *pixbuf_expander_open;
   GdkPixbuf *pixbuf_expander_closed;
 
-  GIcon *gicon;
-
   gboolean follow_state;
 
-  gchar *stock_id;
   gchar *stock_detail;
-  gchar *icon_name;
 };
 
 
@@ -116,8 +109,26 @@ gtk_cell_renderer_pixbuf_init (GtkCellRendererPixbuf *cellpixbuf)
                                                   GTK_TYPE_CELL_RENDERER_PIXBUF,
                                                   GtkCellRendererPixbufPrivate);
   priv = cellpixbuf->priv;
+  priv->icon_helper = _gtk_icon_helper_new ();
+  priv->icon_size = GTK_ICON_SIZE_MENU;
+}
+
+static void
+gtk_cell_renderer_pixbuf_finalize (GObject *object)
+{
+  GtkCellRendererPixbuf *cellpixbuf = GTK_CELL_RENDERER_PIXBUF (object);
+  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
 
-  priv->stock_size = GTK_ICON_SIZE_MENU;
+  g_clear_object (&priv->icon_helper);
+
+  if (priv->pixbuf_expander_open)
+    g_object_unref (priv->pixbuf_expander_open);
+  if (priv->pixbuf_expander_closed)
+    g_object_unref (priv->pixbuf_expander_closed);
+
+  g_free (priv->stock_detail);
+
+  G_OBJECT_CLASS (gtk_cell_renderer_pixbuf_parent_class)->finalize (object);
 }
 
 static void
@@ -239,29 +250,8 @@ gtk_cell_renderer_pixbuf_class_init (GtkCellRendererPixbufClass *class)
 
 
   g_type_class_add_private (object_class, sizeof (GtkCellRendererPixbufPrivate));
-}
 
-static void
-gtk_cell_renderer_pixbuf_finalize (GObject *object)
-{
-  GtkCellRendererPixbuf *cellpixbuf = GTK_CELL_RENDERER_PIXBUF (object);
-  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
-
-  if (priv->pixbuf)
-    g_object_unref (priv->pixbuf);
-  if (priv->pixbuf_expander_open)
-    g_object_unref (priv->pixbuf_expander_open);
-  if (priv->pixbuf_expander_closed)
-    g_object_unref (priv->pixbuf_expander_closed);
-
-  g_free (priv->stock_id);
-  g_free (priv->stock_detail);
-  g_free (priv->icon_name);
-
-  if (priv->gicon)
-    g_object_unref (priv->gicon);
-
-  G_OBJECT_CLASS (gtk_cell_renderer_pixbuf_parent_class)->finalize (object);
+  gtk_cell_renderer_class_set_accessible_type (cell_class, GTK_TYPE_IMAGE_CELL_ACCESSIBLE);
 }
 
 static void
@@ -276,7 +266,7 @@ gtk_cell_renderer_pixbuf_get_property (GObject        *object,
   switch (param_id)
     {
     case PROP_PIXBUF:
-      g_value_set_object (value, priv->pixbuf);
+      g_value_set_object (value, _gtk_icon_helper_peek_pixbuf (priv->icon_helper));
       break;
     case PROP_PIXBUF_EXPANDER_OPEN:
       g_value_set_object (value, priv->pixbuf_expander_open);
@@ -285,10 +275,10 @@ gtk_cell_renderer_pixbuf_get_property (GObject        *object,
       g_value_set_object (value, priv->pixbuf_expander_closed);
       break;
     case PROP_STOCK_ID:
-      g_value_set_string (value, priv->stock_id);
+      g_value_set_string (value, _gtk_icon_helper_get_stock_id (priv->icon_helper));
       break;
     case PROP_STOCK_SIZE:
-      g_value_set_uint (value, priv->stock_size);
+      g_value_set_uint (value, priv->icon_size);
       break;
     case PROP_STOCK_DETAIL:
       g_value_set_string (value, priv->stock_detail);
@@ -297,10 +287,10 @@ gtk_cell_renderer_pixbuf_get_property (GObject        *object,
       g_value_set_boolean (value, priv->follow_state);
       break;
     case PROP_ICON_NAME:
-      g_value_set_string (value, priv->icon_name);
+      g_value_set_string (value, _gtk_icon_helper_get_icon_name (priv->icon_helper));
       break;
     case PROP_GICON:
-      g_value_set_object (value, priv->gicon);
+      g_value_set_object (value, _gtk_icon_helper_peek_gicon (priv->icon_helper));
       break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
@@ -308,6 +298,34 @@ gtk_cell_renderer_pixbuf_get_property (GObject        *object,
     }
 }
 
+static void
+gtk_cell_renderer_pixbuf_reset (GtkCellRendererPixbuf *cellpixbuf)
+{
+  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
+  GtkImageType storage_type = _gtk_icon_helper_get_storage_type (priv->icon_helper);
+
+  switch (storage_type)
+    {
+    case GTK_IMAGE_PIXBUF:
+      g_object_notify (G_OBJECT (cellpixbuf), "pixbuf");
+      break;
+    case GTK_IMAGE_STOCK:
+      g_object_notify (G_OBJECT (cellpixbuf), "stock-id");      
+      break;
+    case GTK_IMAGE_ICON_NAME:
+      g_object_notify (G_OBJECT (cellpixbuf), "icon-name");
+      break;
+    case GTK_IMAGE_GICON:
+      g_object_notify (G_OBJECT (cellpixbuf), "gicon");
+      break;
+    case GTK_IMAGE_EMPTY:
+    default:
+      break;
+    }
+
+  _gtk_icon_helper_clear (priv->icon_helper);
+}
+
 static void
 gtk_cell_renderer_pixbuf_set_property (GObject      *object,
                                       guint         param_id,
@@ -320,30 +338,8 @@ gtk_cell_renderer_pixbuf_set_property (GObject      *object,
   switch (param_id)
     {
     case PROP_PIXBUF:
-      if (priv->pixbuf)
-        g_object_unref (priv->pixbuf);
-      priv->pixbuf = (GdkPixbuf*) g_value_dup_object (value);
-      if (priv->pixbuf)
-        {
-          if (priv->stock_id)
-            {
-              g_free (priv->stock_id);
-              priv->stock_id = NULL;
-              g_object_notify (object, "stock-id");
-            }
-          if (priv->icon_name)
-            {
-              g_free (priv->icon_name);
-              priv->icon_name = NULL;
-              g_object_notify (object, "icon-name");
-            }
-          if (priv->gicon)
-            {
-              g_object_unref (priv->gicon);
-              priv->gicon = NULL;
-              g_object_notify (object, "gicon");
-            }
-        }
+      gtk_cell_renderer_pixbuf_reset (cellpixbuf);
+      _gtk_icon_helper_set_pixbuf (priv->icon_helper, g_value_get_object (value));
       break;
     case PROP_PIXBUF_EXPANDER_OPEN:
       if (priv->pixbuf_expander_open)
@@ -356,116 +352,27 @@ gtk_cell_renderer_pixbuf_set_property (GObject      *object,
       priv->pixbuf_expander_closed = (GdkPixbuf*) g_value_dup_object (value);
       break;
     case PROP_STOCK_ID:
-      if (priv->stock_id)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          g_free (priv->stock_id);
-        }
-      priv->stock_id = g_value_dup_string (value);
-      if (priv->stock_id)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          if (priv->icon_name)
-            {
-              g_free (priv->icon_name);
-              priv->icon_name = NULL;
-              g_object_notify (object, "icon-name");
-            }
-          if (priv->gicon)
-            {
-              g_object_unref (priv->gicon);
-              priv->gicon = NULL;
-              g_object_notify (object, "gicon");
-            }
-        }
+      gtk_cell_renderer_pixbuf_reset (cellpixbuf);
+      _gtk_icon_helper_set_stock_id (priv->icon_helper, g_value_get_string (value), priv->icon_size);
       break;
     case PROP_STOCK_SIZE:
-      priv->stock_size = g_value_get_uint (value);
+      priv->icon_size = g_value_get_uint (value);
+      _gtk_icon_helper_set_icon_size (priv->icon_helper, priv->icon_size);
       break;
     case PROP_STOCK_DETAIL:
       g_free (priv->stock_detail);
       priv->stock_detail = g_value_dup_string (value);
       break;
     case PROP_ICON_NAME:
-      if (priv->icon_name)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          g_free (priv->icon_name);
-        }
-      priv->icon_name = g_value_dup_string (value);
-      if (priv->icon_name)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          if (priv->stock_id)
-            {
-              g_free (priv->stock_id);
-              priv->stock_id = NULL;
-              g_object_notify (object, "stock-id");
-            }
-          if (priv->gicon)
-            {
-              g_object_unref (priv->gicon);
-              priv->gicon = NULL;
-              g_object_notify (object, "gicon");
-            }
-        }
+      gtk_cell_renderer_pixbuf_reset (cellpixbuf);
+      _gtk_icon_helper_set_icon_name (priv->icon_helper, g_value_get_string (value), priv->icon_size);
       break;
     case PROP_FOLLOW_STATE:
       priv->follow_state = g_value_get_boolean (value);
       break;
     case PROP_GICON:
-      if (priv->gicon)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          g_object_unref (priv->gicon);
-        }
-      priv->gicon = (GIcon *) g_value_dup_object (value);
-      if (priv->gicon)
-        {
-          if (priv->pixbuf)
-            {
-              g_object_unref (priv->pixbuf);
-              priv->pixbuf = NULL;
-              g_object_notify (object, "pixbuf");
-            }
-          if (priv->stock_id)
-            {
-              g_free (priv->stock_id);
-              priv->stock_id = NULL;
-              g_object_notify (object, "stock-id");
-            }
-          if (priv->icon_name)
-            {
-              g_free (priv->icon_name);
-              priv->icon_name = NULL;
-              g_object_notify (object, "icon-name");
-            }
-        }
+      gtk_cell_renderer_pixbuf_reset (cellpixbuf);
+      _gtk_icon_helper_set_gicon (priv->icon_helper, g_value_get_object (value), priv->icon_size);
       break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
@@ -492,202 +399,6 @@ gtk_cell_renderer_pixbuf_new (void)
   return g_object_new (GTK_TYPE_CELL_RENDERER_PIXBUF, NULL);
 }
 
-static void
-gtk_cell_renderer_pixbuf_create_stock_pixbuf (GtkCellRendererPixbuf *cellpixbuf,
-                                              GtkWidget             *widget)
-{
-  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
-
-  if (priv->pixbuf)
-    g_object_unref (priv->pixbuf);
-
-  priv->pixbuf = gtk_widget_render_icon_pixbuf (widget,
-                                                priv->stock_id,
-                                                priv->stock_size);
-
-  g_object_notify (G_OBJECT (cellpixbuf), "pixbuf");
-}
-
-static void 
-gtk_cell_renderer_pixbuf_create_themed_pixbuf (GtkCellRendererPixbuf *cellpixbuf,
-                                              GtkWidget             *widget)
-{
-  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
-  GdkScreen *screen;
-  GtkIconTheme *icon_theme;
-  GtkSettings *settings;
-  gint width, height;
-  GtkIconInfo *info;
-
-  if (priv->pixbuf)
-    {
-      g_object_unref (priv->pixbuf);
-      priv->pixbuf = NULL;
-    }
-
-  screen = gtk_widget_get_screen (GTK_WIDGET (widget));
-  icon_theme = gtk_icon_theme_get_for_screen (screen);
-  settings = gtk_settings_get_for_screen (screen);
-
-  if (!gtk_icon_size_lookup_for_settings (settings,
-                                         priv->stock_size,
-                                         &width, &height))
-    {
-      g_warning ("Invalid icon size %u\n", priv->stock_size);
-      width = height = 24;
-    }
-
-  if (priv->icon_name)
-    info = gtk_icon_theme_lookup_icon (icon_theme,
-                                       priv->icon_name,
-                                       MIN (width, height),
-                                       GTK_ICON_LOOKUP_USE_BUILTIN);
-  else if (priv->gicon)
-    info = gtk_icon_theme_lookup_by_gicon (icon_theme,
-                                           priv->gicon,
-                                           MIN (width, height),
-                                           GTK_ICON_LOOKUP_USE_BUILTIN);
-  else
-    info = NULL;
-
-  if (info)
-    {
-      GtkStyleContext *context;
-
-      context = gtk_widget_get_style_context (GTK_WIDGET (widget));
-      priv->pixbuf = gtk_icon_info_load_symbolic_for_context (info,
-                                                              context,
-                                                              NULL,
-                                                              NULL);
-      gtk_icon_info_free (info);
-    }
-
-  g_object_notify (G_OBJECT (cellpixbuf), "pixbuf");
-}
-
-static GdkPixbuf *
-create_symbolic_pixbuf (GtkCellRendererPixbuf *cellpixbuf,
-                       GtkWidget             *widget,
-                        GtkStateFlags          state)
-{
-  GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
-  GdkScreen *screen;
-  GtkIconTheme *icon_theme;
-  GtkSettings *settings;
-  gint width, height;
-  GtkIconInfo *info;
-  GdkPixbuf *pixbuf;
-
-  /* Not a named symbolic icon? */
-  if (priv->icon_name) {
-    if (!g_str_has_suffix (priv->icon_name, "-symbolic"))
-      return NULL;
-  } else if (priv->gicon) {
-    const gchar * const *names;
-    if (!G_IS_THEMED_ICON (priv->gicon))
-      return NULL;
-    names = g_themed_icon_get_names (G_THEMED_ICON (priv->gicon));
-    if (names == NULL || !g_str_has_suffix (names[0], "-symbolic"))
-      return NULL;
-  } else {
-    return NULL;
-  }
-
-  screen = gtk_widget_get_screen (GTK_WIDGET (widget));
-  icon_theme = gtk_icon_theme_get_for_screen (screen);
-  settings = gtk_settings_get_for_screen (screen);
-
-  if (!gtk_icon_size_lookup_for_settings (settings,
-                                         priv->stock_size,
-                                         &width, &height))
-    {
-      g_warning ("Invalid icon size %u\n", priv->stock_size);
-      width = height = 24;
-    }
-
-
-  if (priv->icon_name)
-    info = gtk_icon_theme_lookup_icon (icon_theme,
-                                       priv->icon_name,
-                                       MIN (width, height),
-                                       GTK_ICON_LOOKUP_USE_BUILTIN);
-  else if (priv->gicon)
-    info = gtk_icon_theme_lookup_by_gicon (icon_theme,
-                                           priv->gicon,
-                                           MIN (width, height),
-                                           GTK_ICON_LOOKUP_USE_BUILTIN);
-  else
-    return NULL;
-
-  if (info)
-    {
-      GtkStyleContext *context;
-
-      context = gtk_widget_get_style_context (GTK_WIDGET (widget));
-
-      gtk_style_context_save (context);
-      gtk_style_context_set_state (context, state);
-      pixbuf = gtk_icon_info_load_symbolic_for_context (info,
-                                                        context,
-                                                        NULL,
-                                                        NULL);
-
-      gtk_style_context_restore (context);
-      gtk_icon_info_free (info);
-
-      return pixbuf;
-    }
-
-  return NULL;
-}
-
-static GdkPixbuf *
-create_colorized_pixbuf (GdkPixbuf *src,
-                         GdkRGBA   *new_color)
-{
-  gint i, j;
-  gint width, height, has_alpha, src_row_stride, dst_row_stride;
-  gint red_value, green_value, blue_value;
-  guchar *target_pixels;
-  guchar *original_pixels;
-  guchar *pixsrc;
-  guchar *pixdest;
-  GdkPixbuf *dest;
-
-  red_value = (new_color->red * 65535.0) / 255.0;
-  green_value = (new_color->green * 65535.0) / 255.0;
-  blue_value = (new_color->blue * 65535.0) / 255.0;
-
-  dest = gdk_pixbuf_new (gdk_pixbuf_get_colorspace (src),
-                        gdk_pixbuf_get_has_alpha (src),
-                        gdk_pixbuf_get_bits_per_sample (src),
-                        gdk_pixbuf_get_width (src),
-                        gdk_pixbuf_get_height (src));
-  
-  has_alpha = gdk_pixbuf_get_has_alpha (src);
-  width = gdk_pixbuf_get_width (src);
-  height = gdk_pixbuf_get_height (src);
-  src_row_stride = gdk_pixbuf_get_rowstride (src);
-  dst_row_stride = gdk_pixbuf_get_rowstride (dest);
-  target_pixels = gdk_pixbuf_get_pixels (dest);
-  original_pixels = gdk_pixbuf_get_pixels (src);
-  
-  for (i = 0; i < height; i++) {
-    pixdest = target_pixels + i*dst_row_stride;
-    pixsrc = original_pixels + i*src_row_stride;
-    for (j = 0; j < width; j++) {              
-      *pixdest++ = (*pixsrc++ * red_value) >> 8;
-      *pixdest++ = (*pixsrc++ * green_value) >> 8;
-      *pixdest++ = (*pixsrc++ * blue_value) >> 8;
-      if (has_alpha) {
-       *pixdest++ = *pixsrc++;
-      }
-    }
-  }
-  return dest;
-}
-
-
 static void
 gtk_cell_renderer_pixbuf_get_size (GtkCellRenderer    *cell,
                                   GtkWidget          *widget,
@@ -704,20 +415,19 @@ gtk_cell_renderer_pixbuf_get_size (GtkCellRenderer    *cell,
   gint calc_width;
   gint calc_height;
   gint xpad, ypad;
+  GtkStyleContext *context;
+
+  context = gtk_widget_get_style_context (widget);
+  gtk_style_context_save (context);
+  gtk_style_context_add_class (context, GTK_STYLE_CLASS_IMAGE);
+
+  if (!_gtk_icon_helper_get_is_empty (priv->icon_helper))
+    _gtk_icon_helper_get_size (priv->icon_helper, 
+                               gtk_widget_get_style_context (widget),
+                               &pixbuf_width, &pixbuf_height);
+
+  gtk_style_context_restore (context);
 
-  if (!priv->pixbuf)
-    {
-      if (priv->stock_id)
-       gtk_cell_renderer_pixbuf_create_stock_pixbuf (cellpixbuf, widget);
-      else if (priv->icon_name || priv->gicon)
-       gtk_cell_renderer_pixbuf_create_themed_pixbuf (cellpixbuf, widget);
-    }
-  
-  if (priv->pixbuf)
-    {
-      pixbuf_width  = gdk_pixbuf_get_width (priv->pixbuf);
-      pixbuf_height = gdk_pixbuf_get_height (priv->pixbuf);
-    }
   if (priv->pixbuf_expander_open)
     {
       pixbuf_width  = MAX (pixbuf_width, gdk_pixbuf_get_width (priv->pixbuf_expander_open));
@@ -777,104 +487,70 @@ gtk_cell_renderer_pixbuf_render (GtkCellRenderer      *cell,
   GtkCellRendererPixbuf *cellpixbuf = (GtkCellRendererPixbuf *) cell;
   GtkCellRendererPixbufPrivate *priv = cellpixbuf->priv;
   GtkStyleContext *context;
-  GdkPixbuf *pixbuf;
-  GdkPixbuf *invisible = NULL;
-  GdkPixbuf *colorized = NULL;
-  GdkPixbuf *symbolic = NULL;
   GdkRectangle pix_rect;
   GdkRectangle draw_rect;
   gboolean is_expander;
   gint xpad, ypad;
+  GtkStateFlags state;
+  GtkIconHelper *icon_helper = NULL;
 
   gtk_cell_renderer_pixbuf_get_size (cell, widget, (GdkRectangle *) cell_area,
-                                    &pix_rect.x,
-                                    &pix_rect.y,
-                                    &pix_rect.width,
-                                    &pix_rect.height);
+                                    &pix_rect.x, 
+                                     &pix_rect.y,
+                                     &pix_rect.width,
+                                     &pix_rect.height);
 
   gtk_cell_renderer_get_padding (cell, &xpad, &ypad);
   pix_rect.x += cell_area->x + xpad;
   pix_rect.y += cell_area->y + ypad;
-  pix_rect.width  -= xpad * 2;
+  pix_rect.width -= xpad * 2;
   pix_rect.height -= ypad * 2;
 
   if (!gdk_rectangle_intersect (cell_area, &pix_rect, &draw_rect))
     return;
 
-  pixbuf = priv->pixbuf;
-
-  g_object_get (cell, "is-expander", &is_expander, NULL);
-  if (is_expander)
-    {
-      gboolean is_expanded;
-
-      g_object_get (cell, "is-expanded", &is_expanded, NULL);
-
-      if (is_expanded &&
-         priv->pixbuf_expander_open != NULL)
-       pixbuf = priv->pixbuf_expander_open;
-      else if (!is_expanded &&
-              priv->pixbuf_expander_closed != NULL)
-       pixbuf = priv->pixbuf_expander_closed;
-    }
-
-  if (!pixbuf)
-    return;
-
   context = gtk_widget_get_style_context (widget);
+  gtk_style_context_save (context);
+
+  state = GTK_STATE_FLAG_NORMAL;
 
   if (!gtk_widget_get_sensitive (widget) ||
       !gtk_cell_renderer_get_sensitive (cell))
-    {
-      GtkIconSource *source;
-      
-      source = gtk_icon_source_new ();
-      gtk_icon_source_set_pixbuf (source, pixbuf);
-      /* The size here is arbitrary; since size isn't
-       * wildcarded in the source, it isn't supposed to be
-       * scaled by the engine function
-       */
-      gtk_icon_source_set_size (source, GTK_ICON_SIZE_SMALL_TOOLBAR);
-      gtk_icon_source_set_size_wildcarded (source, FALSE);
-
-      gtk_style_context_save (context);
-      gtk_style_context_set_state (context, GTK_STATE_FLAG_INSENSITIVE);
-
-      pixbuf = invisible = gtk_render_icon_pixbuf (context, source,
-                                                   (GtkIconSize) -1);
-
-      gtk_style_context_restore (context);
-      gtk_icon_source_free (source);
-    }
+    state |= GTK_STATE_FLAG_INSENSITIVE;
   else if (priv->follow_state && 
-          (flags & (GTK_CELL_RENDERER_SELECTED|GTK_CELL_RENDERER_PRELIT)) != 0)
+          (flags & (GTK_CELL_RENDERER_SELECTED |
+                    GTK_CELL_RENDERER_PRELIT)) != 0)
+    state = gtk_cell_renderer_get_state (cell, widget, flags);
+
+  gtk_style_context_set_state (context, state);
+  gtk_style_context_add_class (context, GTK_STYLE_CLASS_IMAGE);
+
+  g_object_get (cell, "is-expander", &is_expander, NULL);
+  if (is_expander)
     {
-      GtkStateFlags state;
+      gboolean is_expanded;
 
-      state = gtk_cell_renderer_get_state (cell, widget, flags);
-      symbolic = create_symbolic_pixbuf (cellpixbuf, widget, state);
+      g_object_get (cell, "is-expanded", &is_expanded, NULL);
 
-      if (!symbolic)
+      if (is_expanded && priv->pixbuf_expander_open != NULL)
         {
-          GdkRGBA color;
-
-          gtk_style_context_get_background_color (context, state, &color);
-          pixbuf = colorized = create_colorized_pixbuf (pixbuf, &color);
+          icon_helper = _gtk_icon_helper_new ();
+          _gtk_icon_helper_set_pixbuf (icon_helper, priv->pixbuf_expander_open);
+        }
+      else if (!is_expanded && priv->pixbuf_expander_closed != NULL)
+        {
+          icon_helper = _gtk_icon_helper_new ();
+          _gtk_icon_helper_set_pixbuf (icon_helper, priv->pixbuf_expander_closed);
         }
-      else
-        pixbuf = symbolic;
     }
 
-  gdk_cairo_set_source_pixbuf (cr, pixbuf, pix_rect.x, pix_rect.y);
-  gdk_cairo_rectangle (cr, &draw_rect);
-  cairo_fill (cr);
-
-  if (invisible)
-    g_object_unref (invisible);
+  if (icon_helper == NULL)
+    icon_helper = g_object_ref (priv->icon_helper);
 
-  if (colorized)
-    g_object_unref (colorized);
+  _gtk_icon_helper_draw (icon_helper,
+                         context, cr,
+                         pix_rect.x, pix_rect.y);
+  g_object_unref (icon_helper);
 
-  if (symbolic)
-    g_object_unref (symbolic);
+  gtk_style_context_restore (context);
 }