Fix performance regression when displaying large images

(bzr r9508.1.54)

4 files changed, 19 insertions, 6 deletions

diff --git a/src/display/cairo-utils.cpp b/src/display/cairo-utils.cpp
index 15fceedae..96219e834 100644
--- a/src/display/cairo-utils.cpp
+++ b/src/display/cairo-utils.cpp
@@ -327,6 +327,14 @@ ink_cairo_pattern_set_matrix(cairo_pattern_t *cp, Geom::Matrix const &m)
 void
 ink_cairo_set_source_argb32_pixbuf(cairo_t *ct, GdkPixbuf *pb, double x, double y)
 {
+    cairo_surface_t *pbs = ink_cairo_surface_create_for_argb32_pixbuf(pb);
+    cairo_set_source_surface(ct, pbs, x, y);
+    cairo_surface_destroy(pbs);
+}
+
+cairo_surface_t *
+ink_cairo_surface_create_for_argb32_pixbuf(GdkPixbuf *pb)
+{
     guchar *data = gdk_pixbuf_get_pixels(pb);
     int w = gdk_pixbuf_get_width(pb);
     int h = gdk_pixbuf_get_height(pb);
@@ -334,8 +342,7 @@ ink_cairo_set_source_argb32_pixbuf(cairo_t *ct, GdkPixbuf *pb, double x, double
 
     cairo_surface_t *pbs = cairo_image_surface_create_for_data(
         data, CAIRO_FORMAT_ARGB32, w, h, stride);
-    cairo_set_source_surface(ct, pbs, x, y);
-    cairo_surface_destroy(pbs);
+    return pbs;
 }
 
 /** @brief Create an exact copy of a surface.
diff --git a/src/display/cairo-utils.h b/src/display/cairo-utils.h
index f74ceed14..0acdcb46a 100644
--- a/src/display/cairo-utils.h
+++ b/src/display/cairo-utils.h
@@ -102,6 +102,7 @@ void convert_pixels_pixbuf_to_argb32(guchar *data, int w, int h, int rs);
 void convert_pixels_argb32_to_pixbuf(guchar *data, int w, int h, int rs);
 void convert_pixbuf_normal_to_argb32(GdkPixbuf *);
 void convert_pixbuf_argb32_to_normal(GdkPixbuf *);
+cairo_surface_t *ink_cairo_surface_create_for_argb32_pixbuf(GdkPixbuf *pb);
 
 G_GNUC_CONST inline guint32
 premul_alpha(guint32 color, guint32 alpha)
diff --git a/src/display/nr-arena-image.cpp b/src/display/nr-arena-image.cpp
index d32b6efe0..5f30e0560 100644
--- a/src/display/nr-arena-image.cpp
+++ b/src/display/nr-arena-image.cpp
@@ -93,8 +93,10 @@ nr_arena_image_finalize (NRObject *object)
 {
     NRArenaImage *image = NR_ARENA_IMAGE (object);
 
-    if (image->pixbuf != NULL)
+    if (image->pixbuf != NULL) {
         g_object_unref(image->pixbuf);
+        cairo_surface_destroy(image->surface);
+    }
     if (image->style)
         sp_style_unref(image->style);
 
@@ -159,7 +161,7 @@ nr_arena_image_render( cairo_t *ct, NRArenaItem *item, NRRectL *area, NRPixBlock
         cairo_translate(ct, image->ox, image->oy);
         cairo_scale(ct, image->sx, image->sy);
 
-        gdk_cairo_set_source_pixbuf(ct, image->pixbuf, 0, 0);
+        cairo_set_source_surface(ct, image->surface, 0, 0);
 
         cairo_matrix_t tt;
         Geom::Matrix total;
@@ -311,7 +313,7 @@ nr_arena_image_rect (NRArenaImage *image)
 /* Utility */
 
 void
-nr_arena_image_set_pixbuf (NRArenaImage *image, GdkPixbuf *pb)
+nr_arena_image_set_argb32_pixbuf (NRArenaImage *image, GdkPixbuf *pb)
 {
     nr_return_if_fail (image != NULL);
     nr_return_if_fail (NR_IS_ARENA_IMAGE (image));
@@ -322,8 +324,10 @@ nr_arena_image_set_pixbuf (NRArenaImage *image, GdkPixbuf *pb)
     }
     if (image->pixbuf != NULL) {
         g_object_unref(image->pixbuf);
+        cairo_surface_destroy(image->surface);
     }
     image->pixbuf = pb;
+    image->surface = pb ? ink_cairo_surface_create_for_argb32_pixbuf(pb) : NULL;
 
     nr_arena_item_request_update (NR_ARENA_ITEM (image), NR_ARENA_ITEM_STATE_ALL, FALSE);
 }
diff --git a/src/display/nr-arena-image.h b/src/display/nr-arena-image.h
index 76ff23c29..bde0d41bd 100644
--- a/src/display/nr-arena-image.h
+++ b/src/display/nr-arena-image.h
@@ -26,6 +26,7 @@ NRType nr_arena_image_get_type (void);
 
 struct NRArenaImage : public NRArenaItem {
     GdkPixbuf *pixbuf;
+    cairo_surface_t *surface;
 
     Geom::Matrix ctm;
     Geom::Rect clipbox;
@@ -45,7 +46,7 @@ struct NRArenaImageClass {
     NRArenaItemClass parent_class;
 };
 
-void nr_arena_image_set_pixbuf (NRArenaImage *image, GdkPixbuf *pb);
+void nr_arena_image_set_argb32_pixbuf (NRArenaImage *image, GdkPixbuf *pb);
 void nr_arena_image_set_style (NRArenaImage *image, SPStyle *style);
 void nr_arena_image_set_clipbox (NRArenaImage *image, Geom::Rect const &clip);
 void nr_arena_image_set_origin (NRArenaImage *image, Geom::Point const &origin);