[Libreoffice-commits] core.git: oox/source

[Gülşah Köse (via logerrit)]

 oox/source/drawingml/fillproperties.cxx |   19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

New commits:
commit 7400372efb3d279225a966a355644d2a3e7d42c3
Author:     Gülşah Köse <gulsah.kose at collabora.com>
AuthorDate: Tue Feb 23 16:41:15 2021 +0300
Commit:     Gülşah Köse <gulsah.kose at collabora.com>
CommitDate: Wed Feb 24 13:18:16 2021 +0100

    Prevent the unnecessary lclCropGraphic call.
    
    In case text::GraphicCrop aGraphCrop( 0, 0, 0, 0 )
    we don't need to call lclCropGraphic.
    
    Change-Id: I9960900f9b5c5b770824c76ffeb9ec9b51cbaef3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111401
    Tested-by: Jenkins
    Reviewed-by: Gülşah Köse <gulsah.kose at collabora.com>

diff --git a/oox/source/drawingml/fillproperties.cxx b/oox/source/drawingml/fillproperties.cxx
index 2022df1d335a..4d9fc2d8688b 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -777,7 +777,8 @@ void FillProperties::pushToPropMap( ShapePropertyMap& rPropMap,
                                 aGraphCrop.Bottom = static_cast< sal_Int32 >( ( static_cast< double >( aOriginalSize.Height ) * aFillRect.Y2 ) / 100000 );
                             rPropMap.setProperty(PROP_GraphicCrop, aGraphCrop);
 
-                            if(bIsCustomShape)
+                            if(bIsCustomShape &&
+                               ( aGraphCrop.Left != 0 || aGraphCrop.Right != 0 || aGraphCrop.Top != 0 || aGraphCrop.Bottom != 0))
                             {
                                 xGraphic = lclCropGraphic(xGraphic, aFillRect);
                                 rPropMap.setProperty(ShapeProperty::FillBitmap, xGraphic);
@@ -883,17 +884,17 @@ void GraphicProperties::pushToPropMap( PropertyMap& rPropMap, const GraphicHelpe
                 if ( oClipRect.Y2 )
                     aGraphCrop.Bottom = rtl::math::round( ( static_cast< double >( aOriginalSize.Height ) * oClipRect.Y2 ) / 100000 );
                 rPropMap.setProperty(PROP_GraphicCrop, aGraphCrop);
-            }
 
-            if(mbIsCustomShape)
-            {
-                geometry::IntegerRectangle2D aCropRect = oClipRect;
-                lclCalculateCropPercentage(xGraphic, aCropRect);
-                xGraphic = lclCropGraphic(xGraphic, aCropRect);
+                if(mbIsCustomShape &&
+                   ( aGraphCrop.Left != 0 || aGraphCrop.Right != 0 || aGraphCrop.Top != 0 || aGraphCrop.Bottom != 0))
+                {
+                    geometry::IntegerRectangle2D aCropRect = oClipRect;
+                    lclCalculateCropPercentage(xGraphic, aCropRect);
+                    xGraphic = lclCropGraphic(xGraphic, aCropRect);
 
-                rPropMap.setProperty(PROP_FillBitmap, xGraphic);
+                    rPropMap.setProperty(PROP_FillBitmap, xGraphic);
+                }
             }
-
         }
 
         if(mbIsCustomShape)