From 6e580f3f53ae2de086a08c8ba1958b67874eb9c5 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <caolanm@redhat.com>
Date: Mon, 14 Jul 2014 12:13:21 +0100
Subject: Related: fdo#52226 ensure graphics are swapped in on
 DrawingML::WriteImage

I imagine it would be best that the Graphics were delivered pre-swapped in by
higher levels in case there are second level caches or more complex caching
systemed wrapped around it, so warn about it in debug mode but give it a
last-ditch shot anyway. i.e. while the .docx problem should be fixed there
is a report of a very similar .xlsx problem

Change-Id: Ie40ee10fe5cba8ff9c321f47b83e33ee2c1425fd
---
 oox/source/export/drawingml.cxx | 34 ++++++++++++++++++++++++++--------
 1 file changed, 26 insertions(+), 8 deletions(-)

(limited to 'oox')

diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index 6cad974774d8..1b35cac7f9cf 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -809,14 +809,32 @@ OUString DrawingML::WriteImage( const Graphic& rGraphic , bool bRelPathToMedia )
             break;
         default: {
             GraphicType aType = rGraphic.GetType();
-            if ( aType == GRAPHIC_BITMAP ) {
-                GraphicConverter::Export( aStream, rGraphic, CVT_PNG );
-                sMediaType = "image/png";
-                pExtension = ".png";
-            } else if ( aType == GRAPHIC_GDIMETAFILE ) {
-                GraphicConverter::Export( aStream, rGraphic, CVT_EMF );
-                sMediaType = "image/x-emf";
-                pExtension = ".emf";
+            if ( aType == GRAPHIC_BITMAP && GRAPHIC_GDIMETAFILE) {
+                bool bSwapped = rGraphic.IsSwapOut();
+
+                //Warn rather than just happily swap in because of the comments
+                //in the sw export filters about needing to go through the
+                //hairy SwGrfNode::SwapIn which we would subvert by swapping in
+                //without it knowing about it, so while those ones are fixed we
+                //probably have to assume that we should ideally be presented
+                //here with already swapped in graphics.
+                SAL_WARN_IF(bSwapped, "oox", "attempted to output swapped out graphic");
+
+                if (bSwapped)
+                    const_cast<Graphic&>(rGraphic).SwapIn();
+
+                if ( aType == GRAPHIC_BITMAP ) {
+                    GraphicConverter::Export( aStream, rGraphic, CVT_PNG );
+                    sMediaType = "image/png";
+                    pExtension = ".png";
+                } else {
+                    GraphicConverter::Export( aStream, rGraphic, CVT_EMF );
+                    sMediaType = "image/x-emf";
+                    pExtension = ".emf";
+                }
+
+                if (bSwapped)
+                    const_cast<Graphic&>(rGraphic).SwapOut();
             } else {
                 OSL_TRACE( "unhandled graphic type" );
                 /*Earlier, even in case of unhandled graphic types we were
-- 
cgit