Fix printing multiple copies of a document

Previously only the first page was printed properly,
all following pages were either blank or were not printed at all.

This was caused by finishing/destroying the cairo surface of the
cairo context associated with the GtkPrintContext used for printing
which made it impossible to draw to it for any consecutive page

Fixed bugs:
  - https://bugs.launchpad.net/inkscape/+bug/1733424
  - https://bugs.launchpad.net/inkscape/+bug/490866

2 files changed, 8 insertions, 5 deletions

diff --git a/src/extension/internal/cairo-render-context.cpp b/src/extension/internal/cairo-render-context.cpp
index 972081c0d..16100dc88 100644
--- a/src/extension/internal/cairo-render-context.cpp
+++ b/src/extension/internal/cairo-render-context.cpp
@@ -878,7 +878,7 @@ CairoRenderContext::_finishSurfaceSetup(cairo_surface_t *surface, cairo_matrix_t
 }
 
 bool
-CairoRenderContext::finish(void)
+CairoRenderContext::finish(bool finish_surface)
 {
     g_assert( _is_valid );
 
@@ -893,10 +893,13 @@ CairoRenderContext::finish(void)
     }
 
     cairo_destroy(_cr);
-    cairo_surface_finish(_surface);
-    cairo_status_t status = cairo_surface_status(_surface);
-    cairo_surface_destroy(_surface);
     _cr = NULL;
+    
+    if (finish_surface)
+        cairo_surface_finish(_surface);
+    cairo_status_t status = cairo_surface_status(_surface);
+    if (finish_surface)
+        cairo_surface_destroy(_surface);
     _surface = NULL;
 
     if (_layout)
diff --git a/src/extension/internal/cairo-render-context.h b/src/extension/internal/cairo-render-context.h
index be5169a74..a0ca2665b 100644
--- a/src/extension/internal/cairo-render-context.h
+++ b/src/extension/internal/cairo-render-context.h
@@ -72,7 +72,7 @@ class CairoRenderContext {
 public:
     CairoRenderContext *cloneMe(void) const;
     CairoRenderContext *cloneMe(double width, double height) const;
-    bool finish(void);
+    bool finish(bool finish_surface = true);
 
     CairoRenderer *getRenderer(void) const;
     cairo_t *getCairoContext(void) const;