stylecontext: Do invalidation on first resize container

[~andy/gtk] / gtk / gtkcssimageurl.c

diff --git a/gtk/gtkcssimageurl.c b/gtk/gtkcssimageurl.c
index 2b51d3558d11c9214c3881113ea31a657265d0a3..0a7c4f3eef2dfb84de5f1cc77f0e57c8b6e61178 100644 (file)
--- a/gtk/gtkcssimageurl.c
+++ b/gtk/gtkcssimageurl.c
@@ -26,12 +26,67 @@
 
 G_DEFINE_TYPE (GtkCssImageUrl, _gtk_css_image_url, GTK_TYPE_CSS_IMAGE)
 
+static GtkCssImage *
+gtk_css_image_url_load_image (GtkCssImageUrl *url)
+{
+  GdkPixbuf *pixbuf;
+  GError *error = NULL;
+  GFileInputStream *input;
+
+  if (url->loaded_image)
+    return url->loaded_image;
+
+  /* We special case resources here so we can use
+     gdk_pixbuf_new_from_resource, which in turn has some special casing
+     for GdkPixdata files to avoid duplicating the memory for the pixbufs */
+  if (g_file_has_uri_scheme (url->file, "resource"))
+    {
+      char *uri = g_file_get_uri (url->file);
+      char *resource_path = g_uri_unescape_string (uri + strlen ("resource://"), NULL);
+
+      pixbuf = gdk_pixbuf_new_from_resource (resource_path, &error);
+      g_free (resource_path);
+      g_free (uri);
+    }
+  else
+    {
+      input = g_file_read (url->file, NULL, &error);
+      if (input != NULL)
+       {
+          pixbuf = gdk_pixbuf_new_from_stream (G_INPUT_STREAM (input), NULL, &error);
+          g_object_unref (input);
+       }
+      else
+        {
+          pixbuf = NULL;
+        }
+    }
+
+  if (pixbuf == NULL)
+    {
+      cairo_surface_t *empty = cairo_image_surface_create (CAIRO_FORMAT_ARGB32, 0, 0);
+
+      /* XXX: Can we get the error somehow sent to the CssProvider?
+       * I don't like just dumping it to stderr or losing it completely. */
+      g_warning ("Error loading image: %s", error->message);
+      g_error_free (error);
+      url->loaded_image = _gtk_css_image_surface_new (empty);
+      cairo_surface_destroy (empty);
+      return url->loaded_image; 
+    }
+
+  url->loaded_image = _gtk_css_image_surface_new_for_pixbuf (pixbuf);
+  g_object_unref (pixbuf);
+
+  return url->loaded_image;
+}
+
 static int
 gtk_css_image_url_get_width (GtkCssImage *image)
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  return _gtk_css_image_get_width (url->loaded_image);
+  return _gtk_css_image_get_width (gtk_css_image_url_load_image (url));
 }
 
 static int
@@ -39,7 +94,7 @@ gtk_css_image_url_get_height (GtkCssImage *image)
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  return _gtk_css_image_get_height (url->loaded_image);
+  return _gtk_css_image_get_height (gtk_css_image_url_load_image (url));
 }
 
 static double
@@ -47,7 +102,7 @@ gtk_css_image_url_get_aspect_ratio (GtkCssImage *image)
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  return _gtk_css_image_get_aspect_ratio (url->loaded_image);
+  return _gtk_css_image_get_aspect_ratio (gtk_css_image_url_load_image (url));
 }
 
 static void
@@ -58,7 +113,7 @@ gtk_css_image_url_draw (GtkCssImage        *image,
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  _gtk_css_image_draw (url->loaded_image, cr, width, height);
+  _gtk_css_image_draw (gtk_css_image_url_load_image (url), cr, width, height);
 }
 
 static GtkCssImage *
@@ -71,7 +126,7 @@ gtk_css_image_url_compute (GtkCssImage             *image,
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  return g_object_ref (url->loaded_image);
+  return g_object_ref (gtk_css_image_url_load_image (url));
 }
 
 static gboolean
@@ -79,48 +134,11 @@ gtk_css_image_url_parse (GtkCssImage  *image,
                          GtkCssParser *parser)
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
-  GdkPixbuf *pixbuf;
-  GError *error = NULL;
-  GFileInputStream *input;
 
   url->file = _gtk_css_parser_read_url (parser);
   if (url->file == NULL)
     return FALSE;
 
-  /* We special case resources here so we can use
-     gdk_pixbuf_new_from_resource, which in turn has some special casing
-     for GdkPixdata files to avoid duplicating the memory for the pixbufs */
-  if (g_file_has_uri_scheme (url->file, "resource"))
-    {
-      char *uri = g_file_get_uri (url->file);
-      char *resource_path = g_uri_unescape_string (uri + strlen ("resource://"), NULL);
-
-      pixbuf = gdk_pixbuf_new_from_resource (resource_path, &error);
-      g_free (resource_path);
-      g_free (uri);
-    }
-  else
-    {
-      input = g_file_read (url->file, NULL, &error);
-      if (input == NULL)
-       {
-         _gtk_css_parser_take_error (parser, error);
-         return FALSE;
-       }
-
-      pixbuf = gdk_pixbuf_new_from_stream (G_INPUT_STREAM (input), NULL, &error);
-      g_object_unref (input);
-    }
-
-  if (pixbuf == NULL)
-    {
-      _gtk_css_parser_take_error (parser, error);
-      return FALSE;
-    }
-
-  url->loaded_image = _gtk_css_image_surface_new_for_pixbuf (pixbuf);
-  g_object_unref (pixbuf);
-
   return TRUE;
 }
 
@@ -130,7 +148,7 @@ gtk_css_image_url_print (GtkCssImage *image,
 {
   GtkCssImageUrl *url = GTK_CSS_IMAGE_URL (image);
 
-  _gtk_css_image_print (url->loaded_image, string);
+  _gtk_css_image_print (gtk_css_image_url_load_image (url), string);
 }
 
 static void