]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkimage.c
stylecontext: Do invalidation on first resize container
[~andy/gtk] / gtk / gtkimage.c
index 7e1e23e4e565ae080145e9f50ed829250e50e6a0..d373b51b709d81586f61af595b9504d836775b3e 100644 (file)
@@ -12,9 +12,7 @@
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser 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 <math.h>
 #include <string.h>
 
 #include "gtkcontainer.h"
-#include "gtkimage.h"
+#include "gtkiconhelperprivate.h"
+#include "gtkimageprivate.h"
 #include "gtkiconfactory.h"
 #include "gtkstock.h"
 #include "gtkicontheme.h"
+#include "gtksizerequest.h"
 #include "gtkintl.h"
 #include "gtkprivate.h"
-#include "gtkalias.h"
+#include "gtktypebuiltins.h"
+
+#include "a11y/gtkimageaccessible.h"
+
+/**
+ * SECTION:gtkimage
+ * @Short_description: A widget displaying an image
+ * @Title: GtkImage
+ * @See_also:#GdkPixbuf
+ *
+ * The #GtkImage widget displays an image. Various kinds of object
+ * can be displayed as an image; most typically, you would load a
+ * #GdkPixbuf ("pixel buffer") from a file, and then display that.
+ * There's a convenience function to do this, gtk_image_new_from_file(),
+ * used as follows:
+ * <informalexample><programlisting>
+ *   GtkWidget *image;
+ *   image = gtk_image_new_from_file ("myfile.png");
+ * </programlisting></informalexample>
+ * If the file isn't loaded successfully, the image will contain a
+ * "broken image" icon similar to that used in many web browsers.
+ * If you want to handle errors in loading the file yourself,
+ * for example by displaying an error message, then load the image with
+ * gdk_pixbuf_new_from_file(), then create the #GtkImage with
+ * gtk_image_new_from_pixbuf().
+ *
+ * The image file may contain an animation, if so the #GtkImage will
+ * display an animation (#GdkPixbufAnimation) instead of a static image.
+ *
+ * #GtkImage is a subclass of #GtkMisc, which implies that you can
+ * align it (center, left, right) and add padding to it, using
+ * #GtkMisc methods.
+ *
+ * #GtkImage is a "no window" widget (has no #GdkWindow of its own),
+ * so by default does not receive events. If you want to receive events
+ * on the image, such as button clicks, place the image inside a
+ * #GtkEventBox, then connect to the event signals on the event box.
+ * <example>
+ * <title>Handling button press events on a
+ * <structname>GtkImage</structname>.</title>
+ * <programlisting>
+ *   static gboolean
+ *   button_press_callback (GtkWidget      *event_box,
+ *                          GdkEventButton *event,
+ *                          gpointer        data)
+ *   {
+ *     g_print ("Event box clicked at coordinates &percnt;f,&percnt;f\n",
+ *              event->x, event->y);
+ *
+ *     /<!---->* Returning TRUE means we handled the event, so the signal
+ *      * emission should be stopped (don't call any further
+ *      * callbacks that may be connected). Return FALSE
+ *      * to continue invoking callbacks.
+ *      *<!---->/
+ *     return TRUE;
+ *   }
+ *
+ *   static GtkWidget*
+ *   create_image (void)
+ *   {
+ *     GtkWidget *image;
+ *     GtkWidget *event_box;
+ *
+ *     image = gtk_image_new_from_file ("myfile.png");
+ *
+ *     event_box = gtk_event_box_new (<!-- -->);
+ *
+ *     gtk_container_add (GTK_CONTAINER (event_box), image);
+ *
+ *     g_signal_connect (G_OBJECT (event_box),
+ *                       "button_press_event",
+ *                       G_CALLBACK (button_press_callback),
+ *                       image);
+ *
+ *     return image;
+ *   }
+ * </programlisting>
+ * </example>
+ *
+ * When handling events on the event box, keep in mind that coordinates
+ * in the image may be different from event box coordinates due to
+ * the alignment and padding settings on the image (see #GtkMisc).
+ * The simplest way to solve this is to set the alignment to 0.0
+ * (left/top), and set the padding to zero. Then the origin of
+ * the image will be the same as the origin of the event box.
+ *
+ * Sometimes an application will want to avoid depending on external data
+ * files, such as image files. GTK+ comes with a program to avoid this,
+ * called <application>gdk-pixbuf-csource</application>. This library
+ * allows you to convert an image into a C variable declaration, which
+ * can then be loaded into a #GdkPixbuf using
+ * gdk_pixbuf_new_from_inline().
+ */
 
-typedef struct _GtkImagePrivate GtkImagePrivate;
 
 struct _GtkImagePrivate
 {
-  /* Only used with GTK_IMAGE_ANIMATION, GTK_IMAGE_PIXBUF */
-  gchar *filename;
+  GtkIconHelper *icon_helper;
 
-  gint pixel_size;
-  guint need_calc_size : 1;
-};
+  gint animation_timeout;
+  GdkPixbufAnimationIter *animation_iter;
 
-#define GTK_IMAGE_GET_PRIVATE(obj) (G_TYPE_INSTANCE_GET_PRIVATE ((obj), GTK_TYPE_IMAGE, GtkImagePrivate))
+  gchar                *filename;       /* Only used with GTK_IMAGE_ANIMATION, GTK_IMAGE_PIXBUF */
+  gchar                *resource_path;  /* Only used with GTK_IMAGE_PIXBUF */
+};
 
 
 #define DEFAULT_ICON_SIZE GTK_ICON_SIZE_BUTTON
-static gint gtk_image_expose       (GtkWidget      *widget,
-                                    GdkEventExpose *event);
-static void gtk_image_unmap        (GtkWidget      *widget);
-static void gtk_image_unrealize    (GtkWidget      *widget);
-static void gtk_image_size_request (GtkWidget      *widget,
-                                    GtkRequisition *requisition);
-static void gtk_image_style_set    (GtkWidget      *widget,
-                                   GtkStyle       *prev_style);
-static void gtk_image_screen_changed (GtkWidget    *widget,
-                                     GdkScreen    *prev_screen);
-static void gtk_image_destroy      (GtkObject      *object);
-static void gtk_image_reset        (GtkImage       *image);
-static void gtk_image_calc_size    (GtkImage       *image);
-
-static void gtk_image_update_size  (GtkImage       *image,
-                                    gint            image_width,
-                                    gint            image_height);
-
-static void gtk_image_set_property      (GObject          *object,
-                                        guint             prop_id,
-                                        const GValue     *value,
-                                        GParamSpec       *pspec);
-static void gtk_image_get_property      (GObject          *object,
-                                        guint             prop_id,
-                                        GValue           *value,
-                                        GParamSpec       *pspec);
-
-static void icon_theme_changed          (GtkImage         *image);
+static gint gtk_image_draw                 (GtkWidget    *widget,
+                                            cairo_t      *cr);
+static void gtk_image_unmap                (GtkWidget    *widget);
+static void gtk_image_unrealize            (GtkWidget    *widget);
+static void gtk_image_get_preferred_width  (GtkWidget    *widget,
+                                            gint         *minimum,
+                                            gint         *natural);
+static void gtk_image_get_preferred_height (GtkWidget    *widget,
+                                            gint         *minimum,
+                                            gint         *natural);
+
+static void gtk_image_style_updated        (GtkWidget    *widget);
+static void gtk_image_screen_changed       (GtkWidget    *widget,
+                                            GdkScreen    *prev_screen);
+static void gtk_image_finalize             (GObject      *object);
+static void gtk_image_reset                (GtkImage     *image);
+
+static void gtk_image_set_property         (GObject      *object,
+                                            guint         prop_id,
+                                            const GValue *value,
+                                            GParamSpec   *pspec);
+static void gtk_image_get_property         (GObject      *object,
+                                            guint         prop_id,
+                                            GValue       *value,
+                                            GParamSpec   *pspec);
+
+static void icon_theme_changed             (GtkImage     *image);
 
 enum
 {
   PROP_0,
   PROP_PIXBUF,
-  PROP_PIXMAP,
-  PROP_IMAGE,
-  PROP_MASK,
   PROP_FILE,
   PROP_STOCK,
   PROP_ICON_SET,
@@ -96,7 +184,9 @@ enum
   PROP_PIXBUF_ANIMATION,
   PROP_ICON_NAME,
   PROP_STORAGE_TYPE,
-  PROP_GICON
+  PROP_GICON,
+  PROP_RESOURCE,
+  PROP_USE_FALLBACK
 };
 
 G_DEFINE_TYPE (GtkImage, gtk_image, GTK_TYPE_MISC)
@@ -105,25 +195,21 @@ static void
 gtk_image_class_init (GtkImageClass *class)
 {
   GObjectClass *gobject_class;
-  GtkObjectClass *object_class;
   GtkWidgetClass *widget_class;
 
   gobject_class = G_OBJECT_CLASS (class);
   
   gobject_class->set_property = gtk_image_set_property;
   gobject_class->get_property = gtk_image_get_property;
-  
-  object_class = GTK_OBJECT_CLASS (class);
-  
-  object_class->destroy = gtk_image_destroy;
+  gobject_class->finalize = gtk_image_finalize;
 
   widget_class = GTK_WIDGET_CLASS (class);
-  
-  widget_class->expose_event = gtk_image_expose;
-  widget_class->size_request = gtk_image_size_request;
+  widget_class->draw = gtk_image_draw;
+  widget_class->get_preferred_width = gtk_image_get_preferred_width;
+  widget_class->get_preferred_height = gtk_image_get_preferred_height;
   widget_class->unmap = gtk_image_unmap;
   widget_class->unrealize = gtk_image_unrealize;
-  widget_class->style_set = gtk_image_style_set;
+  widget_class->style_updated = gtk_image_style_updated;
   widget_class->screen_changed = gtk_image_screen_changed;
   
   g_object_class_install_property (gobject_class,
@@ -134,30 +220,6 @@ gtk_image_class_init (GtkImageClass *class)
                                                         GDK_TYPE_PIXBUF,
                                                         GTK_PARAM_READWRITE));
 
-  g_object_class_install_property (gobject_class,
-                                   PROP_PIXMAP,
-                                   g_param_spec_object ("pixmap",
-                                                        P_("Pixmap"),
-                                                        P_("A GdkPixmap to display"),
-                                                        GDK_TYPE_PIXMAP,
-                                                        GTK_PARAM_READWRITE));
-
-  g_object_class_install_property (gobject_class,
-                                   PROP_IMAGE,
-                                   g_param_spec_object ("image",
-                                                        P_("Image"),
-                                                        P_("A GdkImage to display"),
-                                                        GDK_TYPE_IMAGE,
-                                                        GTK_PARAM_READWRITE));
-
-  g_object_class_install_property (gobject_class,
-                                   PROP_MASK,
-                                   g_param_spec_object ("mask",
-                                                        P_("Mask"),
-                                                        P_("Mask bitmap to use with GdkImage or GdkPixmap"),
-                                                        GDK_TYPE_PIXMAP,
-                                                        GTK_PARAM_READWRITE));
-  
   g_object_class_install_property (gobject_class,
                                    PROP_FILE,
                                    g_param_spec_string ("file",
@@ -249,7 +311,22 @@ gtk_image_class_init (GtkImageClass *class)
                                                         P_("The GIcon being displayed"),
                                                         G_TYPE_ICON,
                                                         GTK_PARAM_READWRITE));
-  
+
+  /**
+   * GtkImage:resource:
+   *
+   * A path to a resource file to display.
+   *
+   * Since: 3.8
+   */
+  g_object_class_install_property (gobject_class,
+                                   PROP_RESOURCE,
+                                   g_param_spec_string ("resource",
+                                                        P_("Resource"),
+                                                        P_("The resource path being displayed"),
+                                                        NULL,
+                                                        GTK_PARAM_READWRITE));
+
   g_object_class_install_property (gobject_class,
                                    PROP_STORAGE_TYPE,
                                    g_param_spec_enum ("storage-type",
@@ -259,34 +336,56 @@ gtk_image_class_init (GtkImageClass *class)
                                                       GTK_IMAGE_EMPTY,
                                                       GTK_PARAM_READABLE));
 
-  g_type_class_add_private (object_class, sizeof (GtkImagePrivate));
+  /**
+   * GtkImage:use-fallback:
+   *
+   * Whether the icon displayed in the GtkImage will use
+   * standard icon names fallback. The value of this property
+   * is only relevant for images of type %GTK_IMAGE_ICON_NAME
+   * and %GTK_IMAGE_GICON.
+   *
+   * Since: 3.0
+   */
+  g_object_class_install_property (gobject_class,
+                                   PROP_USE_FALLBACK,
+                                   g_param_spec_boolean ("use-fallback",
+                                                         P_("Use Fallback"),
+                                                         P_("Whether to use icon names fallback"),
+                                                         FALSE,
+                                                         GTK_PARAM_READWRITE));
+
+  g_type_class_add_private (class, sizeof (GtkImagePrivate));
+
+  gtk_widget_class_set_accessible_type (widget_class, GTK_TYPE_IMAGE_ACCESSIBLE);
 }
 
 static void
 gtk_image_init (GtkImage *image)
 {
-  GtkImagePrivate *priv = GTK_IMAGE_GET_PRIVATE (image);
-
-  GTK_WIDGET_SET_FLAGS (image, GTK_NO_WINDOW);
+  GtkImagePrivate *priv;
 
-  image->storage_type = GTK_IMAGE_EMPTY;
-  image->icon_size = DEFAULT_ICON_SIZE;
-  image->mask = NULL;
+  image->priv = G_TYPE_INSTANCE_GET_PRIVATE (image,
+                                             GTK_TYPE_IMAGE,
+                                             GtkImagePrivate);
+  priv = image->priv;
 
-  priv->pixel_size = -1;
+  gtk_widget_set_has_window (GTK_WIDGET (image), FALSE);
+  priv->icon_helper = _gtk_icon_helper_new ();
 
   priv->filename = NULL;
 }
 
 static void
-gtk_image_destroy (GtkObject *object)
+gtk_image_finalize (GObject *object)
 {
   GtkImage *image = GTK_IMAGE (object);
 
-  gtk_image_reset (image);
+  g_clear_object (&image->priv->icon_helper);
   
-  GTK_OBJECT_CLASS (gtk_image_parent_class)->destroy (object);
-}
+  g_free (image->priv->filename);
+
+  G_OBJECT_CLASS (gtk_image_parent_class)->finalize (object);
+};
 
 static void 
 gtk_image_set_property (GObject      *object,
@@ -294,80 +393,32 @@ gtk_image_set_property (GObject      *object,
                        const GValue *value,
                        GParamSpec   *pspec)
 {
-  GtkImage *image;
+  GtkImage *image = GTK_IMAGE (object);
+  GtkImagePrivate *priv = image->priv;
+  GtkIconSize icon_size = _gtk_icon_helper_get_icon_size (priv->icon_helper);
+
+  if (icon_size == GTK_ICON_SIZE_INVALID)
+    icon_size = DEFAULT_ICON_SIZE;
 
-  image = GTK_IMAGE (object);
-  
   switch (prop_id)
     {
     case PROP_PIXBUF:
       gtk_image_set_from_pixbuf (image,
                                  g_value_get_object (value));
       break;
-    case PROP_PIXMAP:
-      gtk_image_set_from_pixmap (image,
-                                 g_value_get_object (value),
-                                 image->mask);
-      break;
-    case PROP_IMAGE:
-      gtk_image_set_from_image (image,
-                                g_value_get_object (value),
-                                image->mask);
-      break;
-    case PROP_MASK:
-      if (image->storage_type == GTK_IMAGE_PIXMAP)
-        gtk_image_set_from_pixmap (image,
-                                   image->data.pixmap.pixmap,
-                                   g_value_get_object (value));
-      else if (image->storage_type == GTK_IMAGE_IMAGE)
-        gtk_image_set_from_image (image,
-                                  image->data.image.image,
-                                  g_value_get_object (value));
-      else
-        {
-          GdkBitmap *mask;
-
-          mask = g_value_get_object (value);
-
-          if (mask)
-            g_object_ref (mask);
-          
-          gtk_image_clear (image);
-
-          image->mask = mask;
-        }
-      break;
     case PROP_FILE:
       gtk_image_set_from_file (image, g_value_get_string (value));
       break;
     case PROP_STOCK:
       gtk_image_set_from_stock (image, g_value_get_string (value),
-                                image->icon_size);
+                                icon_size);
       break;
     case PROP_ICON_SET:
       gtk_image_set_from_icon_set (image, g_value_get_boxed (value),
-                                   image->icon_size);
+                                   icon_size);
       break;
     case PROP_ICON_SIZE:
-      if (image->storage_type == GTK_IMAGE_STOCK)
-        gtk_image_set_from_stock (image,
-                                  image->data.stock.stock_id,
-                                  g_value_get_int (value));
-      else if (image->storage_type == GTK_IMAGE_ICON_SET)
-        gtk_image_set_from_icon_set (image,
-                                     image->data.icon_set.icon_set,
-                                     g_value_get_int (value));
-      else if (image->storage_type == GTK_IMAGE_ICON_NAME)
-        gtk_image_set_from_icon_name (image,
-                                     image->data.name.icon_name,
-                                     g_value_get_int (value));
-      else if (image->storage_type == GTK_IMAGE_GICON)
-        gtk_image_set_from_gicon (image,
-                                  image->data.gicon.icon,
-                                  g_value_get_int (value));
-      else
-        /* Save to be used when STOCK, ICON_SET, ICON_NAME or GICON property comes in */
-        image->icon_size = g_value_get_int (value);
+      _gtk_icon_helper_set_icon_size (priv->icon_helper, g_value_get_int (value));
       break;
     case PROP_PIXEL_SIZE:
       gtk_image_set_pixel_size (image, g_value_get_int (value));
@@ -378,11 +429,18 @@ gtk_image_set_property (GObject      *object,
       break;
     case PROP_ICON_NAME:
       gtk_image_set_from_icon_name (image, g_value_get_string (value),
-                                   image->icon_size);
+                                   icon_size);
       break;
     case PROP_GICON:
       gtk_image_set_from_gicon (image, g_value_get_object (value),
-                               image->icon_size);
+                               icon_size);
+      break;
+    case PROP_RESOURCE:
+      gtk_image_set_from_resource (image, g_value_get_string (value));
+      break;
+
+    case PROP_USE_FALLBACK:
+      _gtk_icon_helper_set_use_fallback (priv->icon_helper, g_value_get_boolean (value));
       break;
 
     default:
@@ -397,92 +455,47 @@ gtk_image_get_property (GObject     *object,
                        GValue      *value,
                        GParamSpec  *pspec)
 {
-  GtkImage *image;
-  GtkImagePrivate *priv;
-
-  image = GTK_IMAGE (object);
-  priv = GTK_IMAGE_GET_PRIVATE (image);
+  GtkImage *image = GTK_IMAGE (object);
+  GtkImagePrivate *priv = image->priv;
 
-  /* The "getter" functions whine if you try to get the wrong
-   * storage type. This function is instead robust against that,
-   * so that GUI builders don't have to jump through hoops
-   * to avoid g_warning
-   */
-  
   switch (prop_id)
     {
     case PROP_PIXBUF:
-      if (image->storage_type != GTK_IMAGE_PIXBUF)
-        g_value_set_object (value, NULL);
-      else
-        g_value_set_object (value,
-                            gtk_image_get_pixbuf (image));
-      break;
-    case PROP_PIXMAP:
-      if (image->storage_type != GTK_IMAGE_PIXMAP)
-        g_value_set_object (value, NULL);
-      else
-        g_value_set_object (value,
-                            image->data.pixmap.pixmap);
-      break;
-    case PROP_MASK:
-      g_value_set_object (value, image->mask);
-      break;
-    case PROP_IMAGE:
-      if (image->storage_type != GTK_IMAGE_IMAGE)
-        g_value_set_object (value, NULL);
-      else
-        g_value_set_object (value,
-                            image->data.image.image);
+      g_value_set_object (value, _gtk_icon_helper_peek_pixbuf (priv->icon_helper));
       break;
     case PROP_FILE:
       g_value_set_string (value, priv->filename);
       break;
     case PROP_STOCK:
-      if (image->storage_type != GTK_IMAGE_STOCK)
-        g_value_set_string (value, NULL);
-      else
-        g_value_set_string (value,
-                            image->data.stock.stock_id);
+      g_value_set_string (value, _gtk_icon_helper_get_stock_id (priv->icon_helper));
       break;
     case PROP_ICON_SET:
-      if (image->storage_type != GTK_IMAGE_ICON_SET)
-        g_value_set_boxed (value, NULL);
-      else
-        g_value_set_boxed (value,
-                           image->data.icon_set.icon_set);
+      g_value_set_boxed (value, _gtk_icon_helper_peek_icon_set (priv->icon_helper));
       break;      
     case PROP_ICON_SIZE:
-      g_value_set_int (value, image->icon_size);
+      g_value_set_int (value, _gtk_icon_helper_get_icon_size (priv->icon_helper));
       break;
     case PROP_PIXEL_SIZE:
-      g_value_set_int (value, priv->pixel_size);
+      g_value_set_int (value, _gtk_icon_helper_get_pixel_size (priv->icon_helper));
       break;
     case PROP_PIXBUF_ANIMATION:
-      if (image->storage_type != GTK_IMAGE_ANIMATION)
-        g_value_set_object (value, NULL);
-      else
-        g_value_set_object (value,
-                            image->data.anim.anim);
+      g_value_set_object (value, _gtk_icon_helper_peek_animation (priv->icon_helper));
       break;
     case PROP_ICON_NAME:
-      if (image->storage_type != GTK_IMAGE_ICON_NAME)
-       g_value_set_string (value, NULL);
-      else
-       g_value_set_string (value,
-                           image->data.name.icon_name);
+      g_value_set_string (value, _gtk_icon_helper_get_icon_name (priv->icon_helper));
       break;
     case PROP_GICON:
-      if (image->storage_type != GTK_IMAGE_GICON)
-       g_value_set_object (value, NULL);
-      else
-       g_value_set_object (value,
-                           image->data.gicon.icon);
+      g_value_set_object (value, _gtk_icon_helper_peek_gicon (priv->icon_helper));
+      break;
+    case PROP_RESOURCE:
+      g_value_set_string (value, priv->resource_path);
+      break;
+    case PROP_USE_FALLBACK:
+      g_value_set_boolean (value, _gtk_icon_helper_get_use_fallback (priv->icon_helper));
       break;
     case PROP_STORAGE_TYPE:
-      g_value_set_enum (value, image->storage_type);
+      g_value_set_enum (value, _gtk_icon_helper_get_storage_type (priv->icon_helper));
       break;
-      
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
@@ -491,62 +504,45 @@ gtk_image_get_property (GObject     *object,
 
 
 /**
- * gtk_image_new_from_pixmap:
- * @pixmap: (allow-none): a #GdkPixmap, or %NULL
- * @mask: (allow-none): a #GdkBitmap, or %NULL
- *
- * Creates a #GtkImage widget displaying @pixmap with a @mask.
- * A #GdkPixmap is a server-side image buffer in the pixel format of the
- * current display. The #GtkImage does not assume a reference to the
- * pixmap or mask; you still need to unref them if you own references.
- * #GtkImage will add its own reference rather than adopting yours.
+ * gtk_image_new_from_file:
+ * @filename: (type filename): a filename
  * 
- * Return value: a new #GtkImage
- **/
-GtkWidget*
-gtk_image_new_from_pixmap (GdkPixmap *pixmap,
-                           GdkBitmap *mask)
-{
-  GtkImage *image;
-
-  image = g_object_new (GTK_TYPE_IMAGE, NULL);
-
-  gtk_image_set_from_pixmap (image, pixmap, mask);
-
-  return GTK_WIDGET (image);
-}
-
-/**
- * gtk_image_new_from_image:
- * @image: (allow-none): a #GdkImage, or %NULL
- * @mask: (allow-none): a #GdkBitmap, or %NULL
+ * Creates a new #GtkImage displaying the file @filename. If the file
+ * isn't found or can't be loaded, the resulting #GtkImage will
+ * display a "broken image" icon. This function never returns %NULL,
+ * it always returns a valid #GtkImage widget.
  *
- * Creates a #GtkImage widget displaying a @image with a @mask.
- * A #GdkImage is a client-side image buffer in the pixel format of the
- * current display. The #GtkImage does not assume a reference to the
- * image or mask; you still need to unref them if you own references.
- * #GtkImage will add its own reference rather than adopting yours.
+ * If the file contains an animation, the image will contain an
+ * animation.
+ *
+ * If you need to detect failures to load the file, use
+ * gdk_pixbuf_new_from_file() to load the file yourself, then create
+ * the #GtkImage from the pixbuf. (Or for animations, use
+ * gdk_pixbuf_animation_new_from_file()).
+ *
+ * The storage type (gtk_image_get_storage_type()) of the returned
+ * image is not defined, it will be whatever is appropriate for
+ * displaying the file.
  * 
  * Return value: a new #GtkImage
  **/
 GtkWidget*
-gtk_image_new_from_image  (GdkImage  *gdk_image,
-                           GdkBitmap *mask)
+gtk_image_new_from_file   (const gchar *filename)
 {
   GtkImage *image;
 
   image = g_object_new (GTK_TYPE_IMAGE, NULL);
 
-  gtk_image_set_from_image (image, gdk_image, mask);
+  gtk_image_set_from_file (image, filename);
 
   return GTK_WIDGET (image);
 }
 
 /**
- * gtk_image_new_from_file:
- * @filename: a filename
- * 
- * Creates a new #GtkImage displaying the file @filename. If the file
+ * gtk_image_new_from_resource:
+ * @resource_path: a resource path
+ *
+ * Creates a new #GtkImage displaying the resource file @resource_path. If the file
  * isn't found or can't be loaded, the resulting #GtkImage will
  * display a "broken image" icon. This function never returns %NULL,
  * it always returns a valid #GtkImage widget.
@@ -562,17 +558,19 @@ gtk_image_new_from_image  (GdkImage  *gdk_image,
  * The storage type (gtk_image_get_storage_type()) of the returned
  * image is not defined, it will be whatever is appropriate for
  * displaying the file.
- * 
+ *
  * Return value: a new #GtkImage
+ *
+ * Since: 3.4
  **/
 GtkWidget*
-gtk_image_new_from_file   (const gchar *filename)
+gtk_image_new_from_resource (const gchar *resource_path)
 {
   GtkImage *image;
 
   image = g_object_new (GTK_TYPE_IMAGE, NULL);
 
-  gtk_image_set_from_file (image, filename);
+  gtk_image_set_from_resource (image, resource_path);
 
   return GTK_WIDGET (image);
 }
@@ -607,7 +605,7 @@ gtk_image_new_from_pixbuf (GdkPixbuf *pixbuf)
 /**
  * gtk_image_new_from_stock:
  * @stock_id: a stock icon name
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  * 
  * Creates a #GtkImage displaying a stock icon. Sample stock icon
  * names are #GTK_STOCK_OPEN, #GTK_STOCK_QUIT. Sample stock sizes
@@ -634,7 +632,7 @@ gtk_image_new_from_stock (const gchar    *stock_id,
 /**
  * gtk_image_new_from_icon_set:
  * @icon_set: a #GtkIconSet
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  *
  * Creates a #GtkImage displaying an icon set. Sample stock sizes are
  * #GTK_ICON_SIZE_MENU, #GTK_ICON_SIZE_SMALL_TOOLBAR. Instead of using
@@ -696,7 +694,7 @@ gtk_image_new_from_animation (GdkPixbufAnimation *animation)
 /**
  * gtk_image_new_from_icon_name:
  * @icon_name: an icon name
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  * 
  * Creates a #GtkImage displaying an icon from the current icon theme.
  * If the icon name isn't known, a "broken image" icon will be
@@ -723,7 +721,7 @@ gtk_image_new_from_icon_name (const gchar    *icon_name,
 /**
  * gtk_image_new_from_gicon:
  * @icon: an icon
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  * 
  * Creates a #GtkImage displaying an icon from the current icon theme.
  * If the icon name isn't known, a "broken image" icon will be
@@ -748,163 +746,117 @@ gtk_image_new_from_gicon (GIcon *icon,
 }
 
 /**
- * gtk_image_set_from_pixmap:
+ * gtk_image_set_from_file:
  * @image: a #GtkImage
- * @pixmap: (allow-none): a #GdkPixmap or %NULL
- * @mask: (allow-none): a #GdkBitmap or %NULL
+ * @filename: (type filename) (allow-none): a filename or %NULL
  *
- * See gtk_image_new_from_pixmap() for details.
+ * See gtk_image_new_from_file() for details.
  **/
 void
-gtk_image_set_from_pixmap (GtkImage  *image,
-                           GdkPixmap *pixmap,
-                           GdkBitmap *mask)
+gtk_image_set_from_file   (GtkImage    *image,
+                           const gchar *filename)
 {
+  GtkImagePrivate *priv;
+  GdkPixbufAnimation *anim;
+  
   g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (pixmap == NULL ||
-                    GDK_IS_PIXMAP (pixmap));
-  g_return_if_fail (mask == NULL ||
-                    GDK_IS_PIXMAP (mask));
+
+  priv = image->priv;
 
   g_object_freeze_notify (G_OBJECT (image));
   
-  if (pixmap)
-    g_object_ref (pixmap);
-
-  if (mask)
-    g_object_ref (mask);
-
   gtk_image_clear (image);
 
-  image->mask = mask;
-  
-  if (pixmap)
+  if (filename == NULL)
     {
-      int width;
-      int height;
-      
-      image->storage_type = GTK_IMAGE_PIXMAP;
-
-      image->data.pixmap.pixmap = pixmap;
-
-      gdk_drawable_get_size (GDK_DRAWABLE (pixmap), &width, &height);
-
-      gtk_image_update_size (image, width, height);
+      priv->filename = NULL;
+      g_object_thaw_notify (G_OBJECT (image));
+      return;
     }
 
-  g_object_notify (G_OBJECT (image), "pixmap");
-  g_object_notify (G_OBJECT (image), "mask");
-  
-  g_object_thaw_notify (G_OBJECT (image));
-}
-
-/**
- * gtk_image_set_from_image:
- * @image: a #GtkImage
- * @gdk_image: (allow-none): a #GdkImage or %NULL
- * @mask:  (allow-none): a #GdkBitmap or %NULL
- *
- * See gtk_image_new_from_image() for details.
- **/
-void
-gtk_image_set_from_image  (GtkImage  *image,
-                           GdkImage  *gdk_image,
-                           GdkBitmap *mask)
-{
-  g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (gdk_image == NULL ||
-                    GDK_IS_IMAGE (gdk_image));
-  g_return_if_fail (mask == NULL ||
-                    GDK_IS_PIXMAP (mask));
-
-  g_object_freeze_notify (G_OBJECT (image));
-  
-  if (gdk_image)
-    g_object_ref (gdk_image);
-
-  if (mask)
-    g_object_ref (mask);
-
-  gtk_image_clear (image);
+  anim = gdk_pixbuf_animation_new_from_file (filename, NULL);
 
-  if (gdk_image)
+  if (anim == NULL)
     {
-      image->storage_type = GTK_IMAGE_IMAGE;
+      gtk_image_set_from_stock (image,
+                                GTK_STOCK_MISSING_IMAGE,
+                                DEFAULT_ICON_SIZE);
+      g_object_thaw_notify (G_OBJECT (image));
+      return;
+    }
 
-      image->data.image.image = gdk_image;
-      image->mask = mask;
+  /* We could just unconditionally set_from_animation,
+   * but it's nicer for memory if we toss the animation
+   * if it's just a single pixbuf
+   */
 
-      gtk_image_update_size (image, gdk_image->width, gdk_image->height);
-    }
+  if (gdk_pixbuf_animation_is_static_image (anim))
+    gtk_image_set_from_pixbuf (image,
+                              gdk_pixbuf_animation_get_static_image (anim));
   else
-    {
-      /* Clean up the mask if gdk_image was NULL */
-      if (mask)
-        g_object_unref (mask);
-    }
+    gtk_image_set_from_animation (image, anim);
 
-  g_object_notify (G_OBJECT (image), "image");
-  g_object_notify (G_OBJECT (image), "mask");
+  g_object_unref (anim);
+
+  priv->filename = g_strdup (filename);
   
   g_object_thaw_notify (G_OBJECT (image));
 }
 
 /**
- * gtk_image_set_from_file:
+ * gtk_image_set_from_resource:
  * @image: a #GtkImage
- * @filename: (allow-none): a filename or %NULL
+ * @resource_path: (allow-none): a resource path or %NULL
  *
- * See gtk_image_new_from_file() for details.
+ * See gtk_image_new_from_resource() for details.
  **/
 void
-gtk_image_set_from_file   (GtkImage    *image,
-                           const gchar *filename)
+gtk_image_set_from_resource (GtkImage    *image,
+                            const gchar *resource_path)
 {
-  GtkImagePrivate *priv = GTK_IMAGE_GET_PRIVATE (image);
-  GdkPixbufAnimation *anim;
-  
+  GtkImagePrivate *priv;
+  GdkPixbufAnimation *animation;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
-  
+
   gtk_image_clear (image);
 
-  if (filename == NULL)
+  if (resource_path == NULL)
     {
-      priv->filename = NULL;
       g_object_thaw_notify (G_OBJECT (image));
       return;
     }
 
-  anim = gdk_pixbuf_animation_new_from_file (filename, NULL);
+  animation = gdk_pixbuf_animation_new_from_resource (resource_path, NULL);
 
-  if (anim == NULL)
+  if (animation == NULL)
     {
       gtk_image_set_from_stock (image,
                                 GTK_STOCK_MISSING_IMAGE,
-                                GTK_ICON_SIZE_BUTTON);
+                                DEFAULT_ICON_SIZE);
       g_object_thaw_notify (G_OBJECT (image));
       return;
     }
 
-  /* We could just unconditionally set_from_animation,
-   * but it's nicer for memory if we toss the animation
-   * if it's just a single pixbuf
-   */
+  priv->resource_path = g_strdup (resource_path);
 
-  if (gdk_pixbuf_animation_is_static_image (anim))
-    gtk_image_set_from_pixbuf (image,
-                              gdk_pixbuf_animation_get_static_image (anim));
+  if (gdk_pixbuf_animation_is_static_image (animation))
+    gtk_image_set_from_pixbuf (image, gdk_pixbuf_animation_get_static_image (animation));
   else
-    gtk_image_set_from_animation (image, anim);
+    gtk_image_set_from_animation (image, animation);
 
-  g_object_unref (anim);
+  g_object_notify (G_OBJECT (image), "resource");
+
+  g_object_unref (animation);
 
-  priv->filename = g_strdup (filename);
-  
   g_object_thaw_notify (G_OBJECT (image));
 }
 
+
 /**
  * gtk_image_set_from_pixbuf:
  * @image: a #GtkImage
@@ -916,27 +868,20 @@ void
 gtk_image_set_from_pixbuf (GtkImage  *image,
                            GdkPixbuf *pixbuf)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
   g_return_if_fail (pixbuf == NULL ||
                     GDK_IS_PIXBUF (pixbuf));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
   
-  if (pixbuf)
-    g_object_ref (pixbuf);
-
   gtk_image_clear (image);
 
   if (pixbuf != NULL)
-    {
-      image->storage_type = GTK_IMAGE_PIXBUF;
-
-      image->data.pixbuf.pixbuf = pixbuf;
-
-      gtk_image_update_size (image,
-                             gdk_pixbuf_get_width (pixbuf),
-                             gdk_pixbuf_get_height (pixbuf));
-    }
+    _gtk_icon_helper_set_pixbuf (priv->icon_helper, pixbuf);
 
   g_object_notify (G_OBJECT (image), "pixbuf");
   
@@ -947,7 +892,7 @@ gtk_image_set_from_pixbuf (GtkImage  *image,
  * gtk_image_set_from_stock:
  * @image: a #GtkImage
  * @stock_id: a stock icon name
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  *
  * See gtk_image_new_from_stock() for details.
  **/
@@ -956,28 +901,22 @@ gtk_image_set_from_stock  (GtkImage       *image,
                            const gchar    *stock_id,
                            GtkIconSize     size)
 {
+  GtkImagePrivate *priv;
   gchar *new_id;
-  
+
   g_return_if_fail (GTK_IS_IMAGE (image));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
 
-  /* in case stock_id == image->data.stock.stock_id */
   new_id = g_strdup (stock_id);
-  
   gtk_image_clear (image);
 
   if (new_id)
     {
-      image->storage_type = GTK_IMAGE_STOCK;
-      
-      image->data.stock.stock_id = new_id;
-      image->icon_size = size;
-
-      /* Size is demand-computed in size request method
-       * if we're a stock image, since changing the
-       * style impacts the size request
-       */
+      _gtk_icon_helper_set_stock_id (priv->icon_helper, new_id, size);
+      g_free (new_id);
     }
 
   g_object_notify (G_OBJECT (image), "stock");
@@ -990,7 +929,7 @@ gtk_image_set_from_stock  (GtkImage       *image,
  * gtk_image_set_from_icon_set:
  * @image: a #GtkImage
  * @icon_set: a #GtkIconSet
- * @size: a stock icon size
+ * @size: (type int): a stock icon size
  *
  * See gtk_image_new_from_icon_set() for details.
  **/
@@ -999,10 +938,14 @@ gtk_image_set_from_icon_set  (GtkImage       *image,
                               GtkIconSet     *icon_set,
                               GtkIconSize     size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
-  
+
   if (icon_set)
     gtk_icon_set_ref (icon_set);
   
@@ -1010,14 +953,8 @@ gtk_image_set_from_icon_set  (GtkImage       *image,
 
   if (icon_set)
     {      
-      image->storage_type = GTK_IMAGE_ICON_SET;
-      
-      image->data.icon_set.icon_set = icon_set;
-      image->icon_size = size;
-
-      /* Size is demand-computed in size request method
-       * if we're an icon set
-       */
+      _gtk_icon_helper_set_icon_set (priv->icon_helper, icon_set, size);
+      gtk_icon_set_unref (icon_set);
     }
   
   g_object_notify (G_OBJECT (image), "icon-set");
@@ -1038,10 +975,14 @@ void
 gtk_image_set_from_animation (GtkImage           *image,
                               GdkPixbufAnimation *animation)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
   g_return_if_fail (animation == NULL ||
                     GDK_IS_PIXBUF_ANIMATION (animation));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
   
   if (animation)
@@ -1051,15 +992,8 @@ gtk_image_set_from_animation (GtkImage           *image,
 
   if (animation != NULL)
     {
-      image->storage_type = GTK_IMAGE_ANIMATION;
-
-      image->data.anim.anim = animation;
-      image->data.anim.frame_timeout = 0;
-      image->data.anim.iter = NULL;
-      
-      gtk_image_update_size (image,
-                             gdk_pixbuf_animation_get_width (animation),
-                             gdk_pixbuf_animation_get_height (animation));
+      _gtk_icon_helper_set_animation (priv->icon_helper, animation);
+      g_object_unref (animation);
     }
 
   g_object_notify (G_OBJECT (image), "pixbuf-animation");
@@ -1071,7 +1005,7 @@ gtk_image_set_from_animation (GtkImage           *image,
  * gtk_image_set_from_icon_name:
  * @image: a #GtkImage
  * @icon_name: an icon name
- * @size: an icon size
+ * @size: (type int): an icon size
  *
  * See gtk_image_new_from_icon_name() for details.
  * 
@@ -1082,28 +1016,22 @@ gtk_image_set_from_icon_name  (GtkImage       *image,
                               const gchar    *icon_name,
                               GtkIconSize     size)
 {
+  GtkImagePrivate *priv;
   gchar *new_name;
-  
+
   g_return_if_fail (GTK_IS_IMAGE (image));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
 
-  /* in case icon_name == image->data.name.icon_name */
   new_name = g_strdup (icon_name);
-  
   gtk_image_clear (image);
 
   if (new_name)
     {
-      image->storage_type = GTK_IMAGE_ICON_NAME;
-      
-      image->data.name.icon_name = new_name;
-      image->icon_size = size;
-
-      /* Size is demand-computed in size request method
-       * if we're a icon theme image, since changing the
-       * style impacts the size request
-       */
+      _gtk_icon_helper_set_icon_name (priv->icon_helper, new_name, size);
+      g_free (new_name);
     }
 
   g_object_notify (G_OBJECT (image), "icon-name");
@@ -1116,7 +1044,7 @@ gtk_image_set_from_icon_name  (GtkImage       *image,
  * gtk_image_set_from_gicon:
  * @image: a #GtkImage
  * @icon: an icon
- * @size: an icon size
+ * @size: (type int): an icon size
  *
  * See gtk_image_new_from_gicon() for details.
  * 
@@ -1127,27 +1055,23 @@ gtk_image_set_from_gicon  (GtkImage       *image,
                           GIcon          *icon,
                           GtkIconSize     size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
 
+  priv = image->priv;
+
   g_object_freeze_notify (G_OBJECT (image));
 
-  /* in case icon == image->data.gicon.icon */
   if (icon)
     g_object_ref (icon);
-  
+
   gtk_image_clear (image);
 
   if (icon)
     {
-      image->storage_type = GTK_IMAGE_GICON;
-      
-      image->data.gicon.icon = icon;
-      image->icon_size = size;
-
-      /* Size is demand-computed in size request method
-       * if we're a icon theme image, since changing the
-       * style impacts the size request
-       */
+      _gtk_icon_helper_set_gicon (priv->icon_helper, icon, size);
+      g_object_unref (icon);
     }
 
   g_object_notify (G_OBJECT (image), "gicon");
@@ -1171,67 +1095,7 @@ gtk_image_get_storage_type (GtkImage *image)
 {
   g_return_val_if_fail (GTK_IS_IMAGE (image), GTK_IMAGE_EMPTY);
 
-  return image->storage_type;
-}
-
-/**
- * gtk_image_get_pixmap:
- * @image: a #GtkImage
- * @pixmap: (out) (transfer none) (allow-none): location to store the
- *     pixmap, or %NULL
- * @mask: (out) (transfer none) (allow-none): location to store the
- *     mask, or %NULL
- *
- * Gets the pixmap and mask being displayed by the #GtkImage.
- * The storage type of the image must be %GTK_IMAGE_EMPTY or
- * %GTK_IMAGE_PIXMAP (see gtk_image_get_storage_type()).
- * The caller of this function does not own a reference to the
- * returned pixmap and mask.
- **/
-void
-gtk_image_get_pixmap (GtkImage   *image,
-                      GdkPixmap **pixmap,
-                      GdkBitmap **mask)
-{
-  g_return_if_fail (GTK_IS_IMAGE (image)); 
-  g_return_if_fail (image->storage_type == GTK_IMAGE_PIXMAP ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
-  
-  if (pixmap)
-    *pixmap = image->data.pixmap.pixmap;
-  
-  if (mask)
-    *mask = image->mask;
-}
-
-/**
- * gtk_image_get_image:
- * @image: a #GtkImage
- * @gdk_image: (out) (transfer none) (allow-none): return location for
- *     a #GtkImage, or %NULL
- * @mask: (out) (transfer none) (allow-none): return location for a
- *     #GdkBitmap, or %NULL
- * 
- * Gets the #GdkImage and mask being displayed by the #GtkImage.
- * The storage type of the image must be %GTK_IMAGE_EMPTY or
- * %GTK_IMAGE_IMAGE (see gtk_image_get_storage_type()).
- * The caller of this function does not own a reference to the
- * returned image and mask.
- **/
-void
-gtk_image_get_image  (GtkImage   *image,
-                      GdkImage  **gdk_image,
-                      GdkBitmap **mask)
-{
-  g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (image->storage_type == GTK_IMAGE_IMAGE ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
-
-  if (gdk_image)
-    *gdk_image = image->data.image.image;
-  
-  if (mask)
-    *mask = image->mask;
+  return _gtk_icon_helper_get_storage_type (image->priv->icon_helper);
 }
 
 /**
@@ -1251,13 +1115,8 @@ GdkPixbuf*
 gtk_image_get_pixbuf (GtkImage *image)
 {
   g_return_val_if_fail (GTK_IS_IMAGE (image), NULL);
-  g_return_val_if_fail (image->storage_type == GTK_IMAGE_PIXBUF ||
-                        image->storage_type == GTK_IMAGE_EMPTY, NULL);
 
-  if (image->storage_type == GTK_IMAGE_EMPTY)
-    image->data.pixbuf.pixbuf = NULL;
-  
-  return image->data.pixbuf.pixbuf;
+  return _gtk_icon_helper_peek_pixbuf (image->priv->icon_helper);
 }
 
 /**
@@ -1265,7 +1124,8 @@ gtk_image_get_pixbuf (GtkImage *image)
  * @image: a #GtkImage
  * @stock_id: (out) (transfer none) (allow-none): place to store a
  *     stock icon name, or %NULL
- * @size: (out) (allow-none): place to store a stock icon size, or %NULL
+ * @size: (out) (allow-none) (type int): place to store a stock icon
+ *     size, or %NULL
  *
  * Gets the stock icon name and size being displayed by the #GtkImage.
  * The storage type of the image must be %GTK_IMAGE_EMPTY or
@@ -1278,18 +1138,17 @@ gtk_image_get_stock  (GtkImage        *image,
                       gchar          **stock_id,
                       GtkIconSize     *size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (image->storage_type == GTK_IMAGE_STOCK ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
 
-  if (image->storage_type == GTK_IMAGE_EMPTY)
-    image->data.stock.stock_id = NULL;
-  
+  priv = image->priv;
+
   if (stock_id)
-    *stock_id = image->data.stock.stock_id;
+    *stock_id = (gchar *) _gtk_icon_helper_get_stock_id (priv->icon_helper);
 
   if (size)
-    *size = image->icon_size;
+    *size = _gtk_icon_helper_get_icon_size (priv->icon_helper);
 }
 
 /**
@@ -1297,7 +1156,8 @@ gtk_image_get_stock  (GtkImage        *image,
  * @image: a #GtkImage
  * @icon_set: (out) (transfer none) (allow-none): location to store a
  *     #GtkIconSet, or %NULL
- * @size: (out) (allow-none): location to store a stock icon size, or %NULL
+ * @size: (out) (allow-none) (type int): location to store a stock
+ *     icon size, or %NULL
  *
  * Gets the icon set and size being displayed by the #GtkImage.
  * The storage type of the image must be %GTK_IMAGE_EMPTY or
@@ -1308,15 +1168,17 @@ gtk_image_get_icon_set  (GtkImage        *image,
                          GtkIconSet     **icon_set,
                          GtkIconSize     *size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (image->storage_type == GTK_IMAGE_ICON_SET ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
-      
-  if (icon_set)    
-    *icon_set = image->data.icon_set.icon_set;
+
+  priv = image->priv;
+
+  if (icon_set)
+    *icon_set = _gtk_icon_helper_peek_icon_set (priv->icon_helper);
 
   if (size)
-    *size = image->icon_size;
+    *size = _gtk_icon_helper_get_icon_size (priv->icon_helper);
 }
 
 /**
@@ -1335,15 +1197,13 @@ gtk_image_get_icon_set  (GtkImage        *image,
 GdkPixbufAnimation*
 gtk_image_get_animation (GtkImage *image)
 {
+  GtkImagePrivate *priv;
+
   g_return_val_if_fail (GTK_IS_IMAGE (image), NULL);
-  g_return_val_if_fail (image->storage_type == GTK_IMAGE_ANIMATION ||
-                        image->storage_type == GTK_IMAGE_EMPTY,
-                        NULL);
 
-  if (image->storage_type == GTK_IMAGE_EMPTY)
-    image->data.anim.anim = NULL;
-  
-  return image->data.anim.anim;
+  priv = image->priv;
+
+  return _gtk_icon_helper_peek_animation (priv->icon_helper);
 }
 
 /**
@@ -1351,7 +1211,8 @@ gtk_image_get_animation (GtkImage *image)
  * @image: a #GtkImage
  * @icon_name: (out) (transfer none) (allow-none): place to store an
  *     icon name, or %NULL
- * @size: (out) (allow-none): place to store an icon size, or %NULL
+ * @size: (out) (allow-none) (type int): place to store an icon size,
+ *     or %NULL
  *
  * Gets the icon name and size being displayed by the #GtkImage.
  * The storage type of the image must be %GTK_IMAGE_EMPTY or
@@ -1362,22 +1223,21 @@ gtk_image_get_animation (GtkImage *image)
  * Since: 2.6
  **/
 void
-gtk_image_get_icon_name  (GtkImage              *image,
-                         G_CONST_RETURN gchar **icon_name,
-                         GtkIconSize           *size)
+gtk_image_get_icon_name  (GtkImage     *image,
+                         const gchar **icon_name,
+                         GtkIconSize  *size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (image->storage_type == GTK_IMAGE_ICON_NAME ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
 
-  if (image->storage_type == GTK_IMAGE_EMPTY)
-    image->data.name.icon_name = NULL;
-  
+  priv = image->priv;
+
   if (icon_name)
-    *icon_name = image->data.name.icon_name;
+    *icon_name = _gtk_icon_helper_get_icon_name (priv->icon_helper);
 
   if (size)
-    *size = image->icon_size;
+    *size = _gtk_icon_helper_get_icon_size (priv->icon_helper);
 }
 
 /**
@@ -1385,7 +1245,8 @@ gtk_image_get_icon_name  (GtkImage              *image,
  * @image: a #GtkImage
  * @gicon: (out) (transfer none) (allow-none): place to store a
  *     #GIcon, or %NULL
- * @size: (out) (allow-none): place to store an icon size, or %NULL
+ * @size: (out) (allow-none) (type int): place to store an icon size,
+ *     or %NULL
  *
  * Gets the #GIcon and size being displayed by the #GtkImage.
  * The storage type of the image must be %GTK_IMAGE_EMPTY or
@@ -1400,18 +1261,17 @@ gtk_image_get_gicon (GtkImage     *image,
                     GIcon       **gicon,
                     GtkIconSize  *size)
 {
+  GtkImagePrivate *priv;
+
   g_return_if_fail (GTK_IS_IMAGE (image));
-  g_return_if_fail (image->storage_type == GTK_IMAGE_GICON ||
-                    image->storage_type == GTK_IMAGE_EMPTY);
 
-  if (image->storage_type == GTK_IMAGE_EMPTY)
-    image->data.gicon.icon = NULL;
-  
+  priv = image->priv;
+
   if (gicon)
-    *gicon = image->data.gicon.icon;
+    *gicon = _gtk_icon_helper_peek_gicon (priv->icon_helper);
 
   if (size)
-    *size = image->icon_size;
+    *size = _gtk_icon_helper_get_icon_size (priv->icon_helper);
 }
 
 /**
@@ -1427,44 +1287,21 @@ gtk_image_new (void)
   return g_object_new (GTK_TYPE_IMAGE, NULL);
 }
 
-void
-gtk_image_set (GtkImage  *image,
-              GdkImage  *val,
-              GdkBitmap *mask)
-{
-  g_return_if_fail (GTK_IS_IMAGE (image));
-
-  gtk_image_set_from_image (image, val, mask);
-}
-
-void
-gtk_image_get (GtkImage   *image,
-              GdkImage  **val,
-              GdkBitmap **mask)
-{
-  g_return_if_fail (GTK_IS_IMAGE (image));
-
-  gtk_image_get_image (image, val, mask);
-}
-
 static void
 gtk_image_reset_anim_iter (GtkImage *image)
 {
-  if (image->storage_type == GTK_IMAGE_ANIMATION)
+  GtkImagePrivate *priv = image->priv;
+
+  if (gtk_image_get_storage_type (image) == GTK_IMAGE_ANIMATION)
     {
       /* Reset the animation */
-      
-      if (image->data.anim.frame_timeout)
+      if (priv->animation_timeout)
         {
-          g_source_remove (image->data.anim.frame_timeout);
-          image->data.anim.frame_timeout = 0;
+          g_source_remove (priv->animation_timeout);
+          priv->animation_timeout = 0;
         }
 
-      if (image->data.anim.iter)
-        {
-          g_object_unref (image->data.anim.iter);
-          image->data.anim.iter = NULL;
-        }
+      g_clear_object (&priv->animation_iter);
     }
 }
 
@@ -1487,543 +1324,126 @@ gtk_image_unrealize (GtkWidget *widget)
 static gint
 animation_timeout (gpointer data)
 {
-  GtkImage *image;
+  GtkImage *image = GTK_IMAGE (data);
+  GtkImagePrivate *priv = image->priv;
   int delay;
 
-  image = GTK_IMAGE (data);
-  
-  image->data.anim.frame_timeout = 0;
+  priv->animation_timeout = 0;
 
-  gdk_pixbuf_animation_iter_advance (image->data.anim.iter, NULL);
+  gdk_pixbuf_animation_iter_advance (priv->animation_iter, NULL);
 
-  delay = gdk_pixbuf_animation_iter_get_delay_time (image->data.anim.iter);
+  delay = gdk_pixbuf_animation_iter_get_delay_time (priv->animation_iter);
   if (delay >= 0)
     {
-      image->data.anim.frame_timeout =
-        gdk_threads_add_timeout (delay, animation_timeout, image);
+      GtkWidget *widget = GTK_WIDGET (image);
 
-      gtk_widget_queue_draw (GTK_WIDGET (image));
+      priv->animation_timeout =
+        gdk_threads_add_timeout (delay, animation_timeout, image);
 
-      if (GTK_WIDGET_DRAWABLE (image))
-        gdk_window_process_updates (GTK_WIDGET (image)->window, TRUE);
+      gtk_widget_queue_draw (widget);
     }
 
   return FALSE;
 }
 
-static void
-icon_theme_changed (GtkImage *image)
+static GdkPixbuf *
+get_animation_frame (GtkImage *image)
 {
-  if (image->storage_type == GTK_IMAGE_ICON_NAME) 
-    {
-      if (image->data.name.pixbuf)
-       g_object_unref (image->data.name.pixbuf);
-      image->data.name.pixbuf = NULL;
+  GtkImagePrivate *priv = image->priv;
 
-      gtk_widget_queue_draw (GTK_WIDGET (image));
-    }
-  if (image->storage_type == GTK_IMAGE_GICON) 
+  if (priv->animation_iter == NULL)
     {
-      if (image->data.gicon.pixbuf)
-       g_object_unref (image->data.gicon.pixbuf);
-      image->data.gicon.pixbuf = NULL;
+      int delay;
+
+      priv->animation_iter = 
+        gdk_pixbuf_animation_get_iter (_gtk_icon_helper_peek_animation (priv->icon_helper), NULL);
 
-      gtk_widget_queue_draw (GTK_WIDGET (image));
+      delay = gdk_pixbuf_animation_iter_get_delay_time (priv->animation_iter);
+      if (delay >= 0)
+        priv->animation_timeout =
+          gdk_threads_add_timeout (delay, animation_timeout, image);
     }
+
+  /* don't advance the anim iter here, or we could get frame changes between two
+   * exposes of different areas.
+   */
+  return g_object_ref (gdk_pixbuf_animation_iter_get_pixbuf (priv->animation_iter));
 }
 
 static void
-ensure_pixbuf_for_icon_name (GtkImage *image)
+gtk_image_get_preferred_size (GtkImage *image,
+                              gint     *width_out,
+                              gint     *height_out)
 {
-  GtkImagePrivate *priv;
-  GdkScreen *screen;
-  GtkIconTheme *icon_theme;
-  GtkSettings *settings;
+  GtkImagePrivate *priv = image->priv;
   gint width, height;
-  gint *sizes, *s, dist;
-  GtkIconLookupFlags flags;
-  GError *error = NULL;
-
-  g_return_if_fail (image->storage_type == GTK_IMAGE_ICON_NAME);
-
-  priv = GTK_IMAGE_GET_PRIVATE (image);
-  screen = gtk_widget_get_screen (GTK_WIDGET (image));
-  icon_theme = gtk_icon_theme_get_for_screen (screen);
-  settings = gtk_settings_get_for_screen (screen);
-  flags = GTK_ICON_LOOKUP_USE_BUILTIN;
-  if (image->data.name.pixbuf == NULL)
-    {
-      if (priv->pixel_size != -1)
-       {
-         width = height = priv->pixel_size;
-          flags |= GTK_ICON_LOOKUP_FORCE_SIZE;
-       }
-      else if (!gtk_icon_size_lookup_for_settings (settings,
-                                                  image->icon_size,
-                                                  &width, &height))
-       {
-         if (image->icon_size == -1)
-           {
-             /* Find an available size close to 48 */
-             sizes = gtk_icon_theme_get_icon_sizes (icon_theme, image->data.name.icon_name);
-             dist = 100;
-             width = height = 48;
-             for (s = sizes; *s; s++)
-               {
-                 if (*s == -1)
-                   {
-                     width = height = 48;
-                     break;
-                   }
-                 if (*s < 48)
-                   {
-                     if (48 - *s < dist)
-                       {
-                         width = height = *s;
-                         dist = 48 - *s;
-                       }
-                   }
-                 else
-                   {
-                     if (*s - 48 < dist)
-                       {
-                         width = height = *s;
-                         dist = *s - 48;
-                       }
-                   }
-               }
-             g_free (sizes);
-           }
-         else
-           {
-             g_warning ("Invalid icon size %d\n", image->icon_size);
-             width = height = 24;
-           }
-       }
-      image->data.name.pixbuf =
-       gtk_icon_theme_load_icon (icon_theme,
-                                 image->data.name.icon_name,
-                                 MIN (width, height), flags, &error);
-      if (image->data.name.pixbuf == NULL)
-       {
-         g_error_free (error);
-         image->data.name.pixbuf =
-           gtk_widget_render_icon (GTK_WIDGET (image),
-                                   GTK_STOCK_MISSING_IMAGE,
-                                   image->icon_size,
-                                   NULL);
-       }
-    }
+  GtkBorder border;
+  GtkStyleContext *context;
+
+  context = gtk_widget_get_style_context (GTK_WIDGET (image));
+  _gtk_icon_helper_get_size (priv->icon_helper, context, &width, &height);
+  _gtk_misc_get_padding_and_border (GTK_MISC (image), &border);
+
+  width += border.left + border.right;
+  height += border.top + border.bottom;
+
+  if (width_out)
+    *width_out = width;
+  if (height_out)
+    *height_out = height;
 }
 
-static void
-ensure_pixbuf_for_gicon (GtkImage *image)
+static gint
+gtk_image_draw (GtkWidget *widget,
+                cairo_t   *cr)
 {
+  GtkImage *image;
   GtkImagePrivate *priv;
-  GdkScreen *screen;
-  GtkIconTheme *icon_theme;
-  GtkSettings *settings;
-  gint width, height;
-  GtkIconInfo *info;
-  GtkIconLookupFlags flags;
+  GtkMisc *misc;
+  GtkStyleContext *context;
+  gint x, y, width, height;
+  gfloat xalign, yalign;
+  GtkBorder border;
 
-  g_return_if_fail (image->storage_type == GTK_IMAGE_GICON);
+  g_return_val_if_fail (GTK_IS_IMAGE (widget), FALSE);
 
-  priv = GTK_IMAGE_GET_PRIVATE (image);
-  screen = gtk_widget_get_screen (GTK_WIDGET (image));
-  icon_theme = gtk_icon_theme_get_for_screen (screen);
-  settings = gtk_settings_get_for_screen (screen);
-  flags = GTK_ICON_LOOKUP_USE_BUILTIN;
-  if (image->data.gicon.pixbuf == NULL)
-    {
-      if (priv->pixel_size != -1)
-       {
-         width = height = priv->pixel_size;
-          flags |= GTK_ICON_LOOKUP_FORCE_SIZE;
-       }
-      else if (!gtk_icon_size_lookup_for_settings (settings,
-                                                  image->icon_size,
-                                                  &width, &height))
-       {
-         if (image->icon_size == -1)
-           width = height = 48;
-         else
-           {
-             g_warning ("Invalid icon size %d\n", image->icon_size);
-             width = height = 24;
-           }
-       }
-
-      info = gtk_icon_theme_lookup_by_gicon (icon_theme,
-                                            image->data.gicon.icon,
-                                            MIN (width, height), flags);
-      if (info)
-        {
-          image->data.gicon.pixbuf = gtk_icon_info_load_icon (info, NULL);
-          gtk_icon_info_free (info);
-        }
+  image = GTK_IMAGE (widget);
+  misc = GTK_MISC (image);
+  priv = image->priv;
 
-      if (image->data.gicon.pixbuf == NULL)
-       {
-         image->data.gicon.pixbuf =
-           gtk_widget_render_icon (GTK_WIDGET (image),
-                                   GTK_STOCK_MISSING_IMAGE,
-                                   image->icon_size,
-                                   NULL);
-       }
-    }
-}
+  context = gtk_widget_get_style_context (widget);
 
+  gtk_render_background (context, cr,
+                         0, 0,
+                         gtk_widget_get_allocated_width (widget), gtk_widget_get_allocated_height (widget));
+  gtk_render_frame (context, cr,
+                    0, 0,
+                    gtk_widget_get_allocated_width (widget), gtk_widget_get_allocated_height (widget));
 
-/*
- * Like gdk_rectangle_intersect (dest, src, dest), but make 
- * sure that the origin of dest is moved by an "even" offset. 
- * If necessary grow the intersection by one row or column 
- * to achieve this.
- *
- * This is necessary since we can't pass alignment information
- * for the pixelation pattern down to gdk_pixbuf_saturate_and_pixelate(), 
- * thus we have to makesure that the subimages are properly aligned.
- */
-static gboolean
-rectangle_intersect_even (GdkRectangle *src, 
-                         GdkRectangle *dest)
-{
-  gboolean isect;
-  gint x, y;
+  gtk_misc_get_alignment (misc, &xalign, &yalign);
+  gtk_image_get_preferred_size (image, &width, &height);
+  _gtk_misc_get_padding_and_border (misc, &border);
+
+  if (gtk_widget_get_direction (widget) != GTK_TEXT_DIR_LTR)
+    xalign = 1.0 - xalign;
 
-  x = dest->x;
-  y = dest->y;
-  isect = gdk_rectangle_intersect (dest, src, dest);
+  x = floor ((gtk_widget_get_allocated_width (widget) - width) * xalign) + border.left;
+  y = floor ((gtk_widget_get_allocated_height (widget) - height) * yalign) + border.top;
 
-  if ((dest->x - x + dest->y - y) % 2 != 0)
+  if (gtk_image_get_storage_type (image) == GTK_IMAGE_ANIMATION)
     {
-      if (dest->x > x)
-       {
-         dest->x--;
-         dest->width++;
-       }
-      else
-       {
-         dest->y--;
-         dest->height++;
-       }
+      GdkPixbuf *pixbuf = get_animation_frame (image);
+      gtk_render_icon (context, cr,
+                       pixbuf,
+                       x, y);
+      g_object_unref (pixbuf);
     }
-  
-  return isect;
-}
-
-static gint
-gtk_image_expose (GtkWidget      *widget,
-                 GdkEventExpose *event)
-{
-  g_return_val_if_fail (GTK_IS_IMAGE (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-  
-  if (GTK_WIDGET_MAPPED (widget) &&
-      GTK_IMAGE (widget)->storage_type != GTK_IMAGE_EMPTY)
+  else
     {
-      GtkImage *image;
-      GtkMisc *misc;
-      GtkImagePrivate *priv;
-      GdkRectangle area, image_bound;
-      gfloat xalign;
-      gint x, y, mask_x, mask_y;
-      GdkBitmap *mask;
-      GdkPixbuf *pixbuf;
-      gboolean needs_state_transform;
-
-      image = GTK_IMAGE (widget);
-      misc = GTK_MISC (widget);
-      priv = GTK_IMAGE_GET_PRIVATE (image);
-
-      area = event->area;
-
-      /* For stock items and icon sets, we lazily calculate
-       * the size; we might get here between a queue_resize()
-       * and size_request() if something explicitely forces
-       * a redraw.
-       */
-      if (priv->need_calc_size)
-       gtk_image_calc_size (image);
-      
-      if (!gdk_rectangle_intersect (&area, &widget->allocation, &area))
-       return FALSE;
-
-      if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_LTR)
-       xalign = misc->xalign;
-      else
-       xalign = 1.0 - misc->xalign;
-
-      x = floor (widget->allocation.x + misc->xpad
-                + ((widget->allocation.width - widget->requisition.width) * xalign));
-      y = floor (widget->allocation.y + misc->ypad 
-                + ((widget->allocation.height - widget->requisition.height) * misc->yalign));
-      mask_x = x;
-      mask_y = y;
-      
-      image_bound.x = x;
-      image_bound.y = y;      
-      image_bound.width = 0;
-      image_bound.height = 0;      
-
-      mask = NULL;
-      pixbuf = NULL;
-      needs_state_transform = GTK_WIDGET_STATE (widget) != GTK_STATE_NORMAL;
-      
-      switch (image->storage_type)
-        {
-        case GTK_IMAGE_PIXMAP:
-          mask = image->mask;
-          gdk_drawable_get_size (image->data.pixmap.pixmap,
-                                 &image_bound.width,
-                                 &image_bound.height);
-         if (rectangle_intersect_even (&area, &image_bound) &&
-             needs_state_transform)
-            {
-              pixbuf = gdk_pixbuf_get_from_drawable (NULL,
-                                                     image->data.pixmap.pixmap,
-                                                     gtk_widget_get_colormap (widget),
-                                                     image_bound.x - x, image_bound.y - y,
-                                                    0, 0,
-                                                     image_bound.width,
-                                                     image_bound.height);
-
-             x = image_bound.x;
-             y = image_bound.y;
-            }
-         
-          break;
-
-        case GTK_IMAGE_IMAGE:
-          mask = image->mask;
-          image_bound.width = image->data.image.image->width;
-          image_bound.height = image->data.image.image->height;
-
-         if (rectangle_intersect_even (&area, &image_bound) &&
-             needs_state_transform)
-            {
-              pixbuf = gdk_pixbuf_get_from_image (NULL,
-                                                  image->data.image.image,
-                                                  gtk_widget_get_colormap (widget),
-                                                 image_bound.x - x, image_bound.y - y,
-                                                  0, 0,
-                                                  image_bound.width,
-                                                  image_bound.height);
-
-             x = image_bound.x;
-             y = image_bound.y;
-            }
-          break;
-
-        case GTK_IMAGE_PIXBUF:
-          image_bound.width = gdk_pixbuf_get_width (image->data.pixbuf.pixbuf);
-          image_bound.height = gdk_pixbuf_get_height (image->data.pixbuf.pixbuf);            
-
-         if (rectangle_intersect_even (&area, &image_bound) &&
-             needs_state_transform)
-           {
-             pixbuf = gdk_pixbuf_new_subpixbuf (image->data.pixbuf.pixbuf,
-                                                image_bound.x - x, image_bound.y - y,
-                                                image_bound.width, image_bound.height);
-
-             x = image_bound.x;
-             y = image_bound.y;
-           }
-         else
-           {
-             pixbuf = image->data.pixbuf.pixbuf;
-             g_object_ref (pixbuf);
-           }
-          break;
-
-        case GTK_IMAGE_STOCK:
-          pixbuf = gtk_widget_render_icon (widget,
-                                           image->data.stock.stock_id,
-                                           image->icon_size,
-                                           NULL);
-          if (pixbuf)
-            {              
-              image_bound.width = gdk_pixbuf_get_width (pixbuf);
-              image_bound.height = gdk_pixbuf_get_height (pixbuf);
-            }
-
-          /* already done */
-          needs_state_transform = FALSE;
-          break;
-
-        case GTK_IMAGE_ICON_SET:
-          pixbuf =
-            gtk_icon_set_render_icon (image->data.icon_set.icon_set,
-                                      widget->style,
-                                      gtk_widget_get_direction (widget),
-                                      GTK_WIDGET_STATE (widget),
-                                      image->icon_size,
-                                      widget,
-                                      NULL);
-
-          if (pixbuf)
-            {
-              image_bound.width = gdk_pixbuf_get_width (pixbuf);
-              image_bound.height = gdk_pixbuf_get_height (pixbuf);
-            }
-
-          /* already done */
-          needs_state_transform = FALSE;
-          break;
-
-        case GTK_IMAGE_ANIMATION:
-          {
-            if (image->data.anim.iter == NULL)
-              {
-                image->data.anim.iter = gdk_pixbuf_animation_get_iter (image->data.anim.anim, NULL);
-                
-                if (gdk_pixbuf_animation_iter_get_delay_time (image->data.anim.iter) >= 0)
-                  image->data.anim.frame_timeout =
-                    gdk_threads_add_timeout (gdk_pixbuf_animation_iter_get_delay_time (image->data.anim.iter),
-                                   animation_timeout,
-                                   image);
-              }
-
-            image_bound.width = gdk_pixbuf_animation_get_width (image->data.anim.anim);
-            image_bound.height = gdk_pixbuf_animation_get_height (image->data.anim.anim);
-                  
-            /* don't advance the anim iter here, or we could get frame changes between two
-             * exposes of different areas.
-             */
-            
-            pixbuf = gdk_pixbuf_animation_iter_get_pixbuf (image->data.anim.iter);
-            g_object_ref (pixbuf);
-          }
-          break;
-
-       case GTK_IMAGE_ICON_NAME:
-         ensure_pixbuf_for_icon_name (image);
-         pixbuf = image->data.name.pixbuf;
-         if (pixbuf)
-           {
-             g_object_ref (pixbuf);
-             image_bound.width = gdk_pixbuf_get_width (pixbuf);
-             image_bound.height = gdk_pixbuf_get_height (pixbuf);
-           }
-         break;
-
-       case GTK_IMAGE_GICON:
-         ensure_pixbuf_for_gicon (image);
-         pixbuf = image->data.gicon.pixbuf;
-         if (pixbuf)
-           {
-             g_object_ref (pixbuf);
-             image_bound.width = gdk_pixbuf_get_width (pixbuf);
-             image_bound.height = gdk_pixbuf_get_height (pixbuf);
-           }
-         break;
-         
-        case GTK_IMAGE_EMPTY:
-          g_assert_not_reached ();
-          break;
-        }
-
-      if (mask)
-       {
-         gdk_gc_set_clip_mask (widget->style->black_gc, mask);
-         gdk_gc_set_clip_origin (widget->style->black_gc, mask_x, mask_y);
-       }
-
-      if (rectangle_intersect_even (&area, &image_bound))
-        {
-          if (pixbuf)
-            {
-              if (needs_state_transform)
-                {
-                  GtkIconSource *source;
-                  GdkPixbuf *rendered;
-
-                  source = gtk_icon_source_new ();
-                  gtk_icon_source_set_pixbuf (source, pixbuf);
-                  /* The size here is arbitrary; since size isn't
-                   * wildcarded in the souce, 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);
-                  
-                  rendered = gtk_style_render_icon (widget->style,
-                                                    source,
-                                                    gtk_widget_get_direction (widget),
-                                                    GTK_WIDGET_STATE (widget),
-                                                    /* arbitrary */
-                                                    (GtkIconSize)-1,
-                                                    widget,
-                                                    "gtk-image");
-
-                  gtk_icon_source_free (source);
-
-                  g_object_unref (pixbuf);
-                  pixbuf = rendered;
-                }
-
-              if (pixbuf)
-                {
-                  gdk_draw_pixbuf (widget->window,
-                                  widget->style->black_gc,
-                                  pixbuf,
-                                  image_bound.x - x,
-                                  image_bound.y - y,
-                                  image_bound.x,
-                                  image_bound.y,
-                                  image_bound.width,
-                                  image_bound.height,
-                                  GDK_RGB_DITHER_NORMAL,
-                                  0, 0);
-                }
-            }
-          else
-            {
-              switch (image->storage_type)
-                {
-                case GTK_IMAGE_PIXMAP:
-                  gdk_draw_drawable (widget->window,
-                                     widget->style->black_gc,
-                                     image->data.pixmap.pixmap,
-                                     image_bound.x - x, image_bound.y - y,
-                                     image_bound.x, image_bound.y,
-                                     image_bound.width, image_bound.height);
-                  break;
-              
-                case GTK_IMAGE_IMAGE:
-                  gdk_draw_image (widget->window,
-                                  widget->style->black_gc,
-                                  image->data.image.image,
-                                  image_bound.x - x, image_bound.y - y,
-                                  image_bound.x, image_bound.y,
-                                  image_bound.width, image_bound.height);
-                  break;
-
-                case GTK_IMAGE_PIXBUF:
-                case GTK_IMAGE_STOCK:
-                case GTK_IMAGE_ICON_SET:
-                case GTK_IMAGE_ANIMATION:
-               case GTK_IMAGE_ICON_NAME:
-                case GTK_IMAGE_EMPTY:
-               case GTK_IMAGE_GICON:
-                  g_assert_not_reached ();
-                  break;
-                }
-            }
-        } /* if rectangle intersects */      
-
-      if (mask)
-        {
-          gdk_gc_set_clip_mask (widget->style->black_gc, NULL);
-          gdk_gc_set_clip_origin (widget->style->black_gc, 0, 0);
-        }
-      
-      if (pixbuf)
-       g_object_unref (pixbuf);
-
-    } /* if widget is drawable */
+      _gtk_icon_helper_draw (priv->icon_helper, 
+                             context, cr,
+                             x, y);
+    }
 
   return FALSE;
 }
@@ -2031,114 +1451,41 @@ gtk_image_expose (GtkWidget      *widget,
 static void
 gtk_image_reset (GtkImage *image)
 {
-  GtkImagePrivate *priv;
-
-  priv = GTK_IMAGE_GET_PRIVATE (image);
+  GtkImagePrivate *priv = image->priv;
+  GtkImageType storage_type;
 
   g_object_freeze_notify (G_OBJECT (image));
+  storage_type = gtk_image_get_storage_type (image);
   
-  if (image->storage_type != GTK_IMAGE_EMPTY)
+  if (storage_type != GTK_IMAGE_EMPTY)
     g_object_notify (G_OBJECT (image), "storage-type");
 
-  if (image->mask)
-    {
-      g_object_unref (image->mask);
-      image->mask = NULL;
-      g_object_notify (G_OBJECT (image), "mask");
-    }
-
-  if (image->icon_size != DEFAULT_ICON_SIZE)
-    {
-      image->icon_size = DEFAULT_ICON_SIZE;
-      g_object_notify (G_OBJECT (image), "icon-size");
-    }
+  g_object_notify (G_OBJECT (image), "icon-size");
   
-  switch (image->storage_type)
+  switch (storage_type)
     {
-    case GTK_IMAGE_PIXMAP:
-
-      if (image->data.pixmap.pixmap)
-        g_object_unref (image->data.pixmap.pixmap);
-      image->data.pixmap.pixmap = NULL;
-      
-      g_object_notify (G_OBJECT (image), "pixmap");
-      
-      break;
-
-    case GTK_IMAGE_IMAGE:
-
-      if (image->data.image.image)
-        g_object_unref (image->data.image.image);
-      image->data.image.image = NULL;
-      
-      g_object_notify (G_OBJECT (image), "image");
-      
-      break;
-
     case GTK_IMAGE_PIXBUF:
-
-      if (image->data.pixbuf.pixbuf)
-        g_object_unref (image->data.pixbuf.pixbuf);
-
       g_object_notify (G_OBJECT (image), "pixbuf");
-      
       break;
-
     case GTK_IMAGE_STOCK:
-
-      g_free (image->data.stock.stock_id);
-
-      image->data.stock.stock_id = NULL;
-      
       g_object_notify (G_OBJECT (image), "stock");      
       break;
-
     case GTK_IMAGE_ICON_SET:
-      if (image->data.icon_set.icon_set)
-        gtk_icon_set_unref (image->data.icon_set.icon_set);
-      image->data.icon_set.icon_set = NULL;
-      
       g_object_notify (G_OBJECT (image), "icon-set");      
       break;
-
     case GTK_IMAGE_ANIMATION:
-      gtk_image_reset_anim_iter (image);
-      
-      if (image->data.anim.anim)
-        g_object_unref (image->data.anim.anim);
-      image->data.anim.anim = NULL;
-      
+      gtk_image_reset_anim_iter (image);      
       g_object_notify (G_OBJECT (image), "pixbuf-animation");
-      
       break;
-
     case GTK_IMAGE_ICON_NAME:
-      g_free (image->data.name.icon_name);
-      image->data.name.icon_name = NULL;
-      if (image->data.name.pixbuf)
-       g_object_unref (image->data.name.pixbuf);
-      image->data.name.pixbuf = NULL;
-
       g_object_notify (G_OBJECT (image), "icon-name");
-
       break;
-      
     case GTK_IMAGE_GICON:
-      if (image->data.gicon.icon)
-       g_object_unref (image->data.gicon.icon);
-      image->data.gicon.icon = NULL;
-      if (image->data.gicon.pixbuf)
-       g_object_unref (image->data.gicon.pixbuf);
-      image->data.gicon.pixbuf = NULL;
-
       g_object_notify (G_OBJECT (image), "gicon");
-
       break;
-      
     case GTK_IMAGE_EMPTY:
     default:
       break;
-      
     }
 
   if (priv->filename)
@@ -2148,9 +1495,14 @@ gtk_image_reset (GtkImage *image)
       g_object_notify (G_OBJECT (image), "file");
     }
 
-  image->storage_type = GTK_IMAGE_EMPTY;
+  if (priv->resource_path)
+    {
+      g_free (priv->resource_path);
+      priv->resource_path = NULL;
+      g_object_notify (G_OBJECT (image), "resource");
+    }
 
-  memset (&image->data, '\0', sizeof (image->data));
+  _gtk_icon_helper_clear (priv->icon_helper);
 
   g_object_thaw_notify (G_OBJECT (image));
 }
@@ -2166,99 +1518,49 @@ gtk_image_reset (GtkImage *image)
 void
 gtk_image_clear (GtkImage *image)
 {
-  GtkImagePrivate *priv;
-
-  priv = GTK_IMAGE_GET_PRIVATE (image);
-
-  priv->need_calc_size = 1;
-
   gtk_image_reset (image);
-  gtk_image_update_size (image, 0, 0);
+
+  if (gtk_widget_get_visible (GTK_WIDGET (image)))
+      gtk_widget_queue_resize (GTK_WIDGET (image));
 }
 
 static void
-gtk_image_calc_size (GtkImage *image)
+gtk_image_get_preferred_width (GtkWidget *widget,
+                               gint      *minimum,
+                               gint      *natural)
 {
-  GtkWidget *widget = GTK_WIDGET (image);
-  GdkPixbuf *pixbuf = NULL;
-  GtkImagePrivate *priv;
-
-  priv = GTK_IMAGE_GET_PRIVATE (image);
+  gint width;
 
-  priv->need_calc_size = 0;
+  gtk_image_get_preferred_size (GTK_IMAGE (widget), &width, NULL);
+  *minimum = *natural = width;
+} 
 
-  /* We update stock/icon set on every size request, because
-   * the theme could have affected the size; for other kinds of
-   * image, we just update the requisition when the image data
-   * is set.
-   */
-  switch (image->storage_type)
-    {
-    case GTK_IMAGE_STOCK:
-      pixbuf = gtk_widget_render_icon (widget,
-                                      image->data.stock.stock_id,
-                                       image->icon_size,
-                                       NULL);
-      break;
-      
-    case GTK_IMAGE_ICON_SET:
-      pixbuf = gtk_icon_set_render_icon (image->data.icon_set.icon_set,
-                                         widget->style,
-                                         gtk_widget_get_direction (widget),
-                                         GTK_WIDGET_STATE (widget),
-                                         image->icon_size,
-                                         widget,
-                                         NULL);
-      break;
-    case GTK_IMAGE_ICON_NAME:
-      ensure_pixbuf_for_icon_name (image);
-      pixbuf = image->data.name.pixbuf;
-      if (pixbuf) g_object_ref (pixbuf);
-      break;
-    case GTK_IMAGE_GICON:
-      ensure_pixbuf_for_gicon (image);
-      pixbuf = image->data.gicon.pixbuf;
-      if (pixbuf)
-       g_object_ref (pixbuf);
-      break;
-    default:
-      break;
-    }
-
-  if (pixbuf)
-    {
-      widget->requisition.width = gdk_pixbuf_get_width (pixbuf) + GTK_MISC (image)->xpad * 2;
-      widget->requisition.height = gdk_pixbuf_get_height (pixbuf) + GTK_MISC (image)->ypad * 2;
+static void
+gtk_image_get_preferred_height (GtkWidget *widget,
+                                gint      *minimum,
+                                gint      *natural)
+{
+  gint height;
 
-      g_object_unref (pixbuf);
-    }
+  gtk_image_get_preferred_size (GTK_IMAGE (widget), NULL, &height);
+  *minimum = *natural = height;
 }
 
 static void
-gtk_image_size_request (GtkWidget      *widget,
-                        GtkRequisition *requisition)
+icon_theme_changed (GtkImage *image)
 {
-  GtkImage *image;
-  
-  image = GTK_IMAGE (widget);
-
-  gtk_image_calc_size (image);
+  GtkImagePrivate *priv = image->priv;
 
-  /* Chain up to default that simply reads current requisition */
-  GTK_WIDGET_CLASS (gtk_image_parent_class)->size_request (widget, requisition);
+  _gtk_icon_helper_invalidate (priv->icon_helper);
+  gtk_widget_queue_draw (GTK_WIDGET (image));
 }
 
 static void
-gtk_image_style_set (GtkWidget      *widget,
-                    GtkStyle       *prev_style)
+gtk_image_style_updated (GtkWidget *widget)
 {
-  GtkImage *image;
-
-  image = GTK_IMAGE (widget);
-
-  GTK_WIDGET_CLASS (gtk_image_parent_class)->style_set (widget, prev_style);
+  GTK_WIDGET_CLASS (gtk_image_parent_class)->style_updated (widget);
 
-  icon_theme_changed (image);
+  icon_theme_changed (GTK_IMAGE (widget));
 }
 
 static void
@@ -2275,20 +1577,21 @@ gtk_image_screen_changed (GtkWidget *widget,
   icon_theme_changed (image);
 }
 
-
-static void
-gtk_image_update_size (GtkImage *image,
-                       gint      image_width,
-                       gint      image_height)
+void
+_gtk_image_gicon_data_clear (GtkImageGIconData *data)
 {
-  GTK_WIDGET (image)->requisition.width = image_width + GTK_MISC (image)->xpad * 2;
-  GTK_WIDGET (image)->requisition.height = image_height + GTK_MISC (image)->ypad * 2;
-
-  if (GTK_WIDGET_VISIBLE (image))
-    gtk_widget_queue_resize (GTK_WIDGET (image));
+  if (data->pixbuf)
+    {
+      g_object_unref (data->pixbuf);
+      data->pixbuf = NULL;
+    }
+  if (data->icon)
+    {
+      g_object_unref (data->icon);
+      data->icon = NULL;
+    }
 }
 
-
 /**
  * gtk_image_set_pixel_size:
  * @image: a #GtkImage
@@ -2305,37 +1608,20 @@ gtk_image_set_pixel_size (GtkImage *image,
                          gint      pixel_size)
 {
   GtkImagePrivate *priv;
+  gint old_pixel_size;
 
   g_return_if_fail (GTK_IS_IMAGE (image));
-  
-  priv = GTK_IMAGE_GET_PRIVATE (image);
 
-  if (priv->pixel_size != pixel_size)
+  priv = image->priv;
+  old_pixel_size = gtk_image_get_pixel_size (image);
+
+  if (pixel_size != old_pixel_size)
     {
-      priv->pixel_size = pixel_size;
-      
-      if (image->storage_type == GTK_IMAGE_ICON_NAME)
-       {
-         if (image->data.name.pixbuf)
-           {
-             g_object_unref (image->data.name.pixbuf);
-             image->data.name.pixbuf = NULL;
-           }
-         
-         gtk_image_update_size (image, pixel_size, pixel_size);
-       }
-      
-      if (image->storage_type == GTK_IMAGE_GICON)
-       {
-         if (image->data.gicon.pixbuf)
-           {
-             g_object_unref (image->data.gicon.pixbuf);
-             image->data.gicon.pixbuf = NULL;
-           }
-         
-         gtk_image_update_size (image, pixel_size, pixel_size);
-       }
-      
+      _gtk_icon_helper_set_pixel_size (priv->icon_helper, pixel_size);
+
+      if (gtk_widget_get_visible (GTK_WIDGET (image)))
+        gtk_widget_queue_resize (GTK_WIDGET (image));
+
       g_object_notify (G_OBJECT (image), "pixel-size");
     }
 }
@@ -2353,46 +1639,7 @@ gtk_image_set_pixel_size (GtkImage *image,
 gint
 gtk_image_get_pixel_size (GtkImage *image)
 {
-  GtkImagePrivate *priv;
-
   g_return_val_if_fail (GTK_IS_IMAGE (image), -1);
-  
-  priv = GTK_IMAGE_GET_PRIVATE (image);
-
-  return priv->pixel_size;
-}
-
-#if defined (G_OS_WIN32) && !defined (_WIN64)
-
-#undef gtk_image_new_from_file
-
-GtkWidget*
-gtk_image_new_from_file   (const gchar *filename)
-{
-  gchar *utf8_filename = g_locale_to_utf8 (filename, -1, NULL, NULL, NULL);
-  GtkWidget *retval;
-
-  retval = gtk_image_new_from_file_utf8 (utf8_filename);
-
-  g_free (utf8_filename);
-
-  return retval;
-}
-
-#undef gtk_image_set_from_file
-
-void
-gtk_image_set_from_file   (GtkImage    *image,
-                           const gchar *filename)
-{
-  gchar *utf8_filename = g_locale_to_utf8 (filename, -1, NULL, NULL, NULL);
 
-  gtk_image_set_from_file_utf8 (image, utf8_filename);
-
-  g_free (utf8_filename);
+  return _gtk_icon_helper_get_pixel_size (image->priv->icon_helper);
 }
-
-#endif
-
-#define __GTK_IMAGE_C__
-#include "gtkaliasdef.c"