[Libreoffice-commits] core.git: sw/qa sw/source

[Mike Kaganski]

 sw/qa/extras/ooxmlimport/data/tdf92157.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx    |    5 +++++
 sw/source/core/layout/atrfrm.cxx            |    5 +----
 3 files changed, 6 insertions(+), 4 deletions(-)

New commits:
commit 654f6ff28d7a148950b48ed8905d8f13a015a5b5
Author: Mike Kaganski <mike.kaganski at collabora.com>
Date:   Sat Jan 9 18:39:26 2016 +1000

    tdf#92157: allow both dimensions of a graphic to be 0
    
    Commit ca80f73 made it possible for one of dimensions to be zero.
    This commit goes further, allowing opening real-life documents
    with graphics having both width and height equal to 0.
    Thanks to libreoffice at arbruijn.dds.nl for debugging and initial
    patch!
    
    Change-Id: I96a13b776adfd9fe46fc2c7691eb7904400c20a1
    Reviewed-on: https://gerrit.libreoffice.org/21287
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Chris Sherlock <chris.sherlock79 at gmail.com>

diff --git a/sw/qa/extras/ooxmlimport/data/tdf92157.docx b/sw/qa/extras/ooxmlimport/data/tdf92157.docx
new file mode 100644
index 0000000..ba5bc2a
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf92157.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index df0f695..57507ea 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -2980,6 +2980,11 @@ DECLARE_OOXMLIMPORT_TEST(testTdf95775, "tdf95775.docx")
     // This must not fail in layout
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf92157, "tdf92157.docx")
+{
+    // A graphic with dimensions 0,0 should not fail on load
+}
+
 DECLARE_OOXMLIMPORT_TEST(testTdf60351, "tdf60351.docx")
 {
     // Get the first image in the document and check its contour polygon.
diff --git a/sw/source/core/layout/atrfrm.cxx b/sw/source/core/layout/atrfrm.cxx
index 9e0b50c..e0f9418 100644
--- a/sw/source/core/layout/atrfrm.cxx
+++ b/sw/source/core/layout/atrfrm.cxx
@@ -282,10 +282,7 @@ bool SwFormatFrameSize::PutValue( const uno::Any& rVal, sal_uInt8 nMemberId )
                     aTmp.Height() = convertMm100ToTwip(aTmp.Height());
                     aTmp.Width() = convertMm100ToTwip(aTmp.Width());
                 }
-                if(aTmp.Height() || aTmp.Width())
-                    m_aSize = aTmp;
-                else
-                    bRet = false;
+                m_aSize = aTmp;
             }
         }
         break;