#i105065# speedup 3D/FontWork

1 files changed, 40 insertions, 31 deletions

diff --git a/svx/source/customshapes/EnhancedCustomShape2d.cxx b/svx/source/customshapes/EnhancedCustomShape2d.cxx
index 804db0c58a16..0ef28d7972dc 100644
--- a/svx/source/customshapes/EnhancedCustomShape2d.cxx
+++ b/svx/source/customshapes/EnhancedCustomShape2d.cxx
@@ -696,7 +696,16 @@ EnhancedCustomShape2d::EnhancedCustomShape2d( SdrObject* pAObj ) :
 
     ClearItem( SDRATTR_TEXTDIRECTION ); //SJ: vertical writing is not required, by removing this item no outliner is created
 
-    // For primitive rendering, shadow handling is done completely based on the geometry, so i removed it here
+    // #i105323# For 2D AtoShapes, the shadow attirbute does not need to be applied to any
+    // of the constucted helper SdrObjects. This would lead to problems since the shadow
+    // of one helper object would fall on one helper object behind it (e.g. with the
+    // eyes of the smiley shape). This is not wanted; instead a single shadow 'behind'
+    // the AutoShape visualisation is wanted. This is done with primitive functionailty
+    // now in SdrCustomShapePrimitive2D::create2DDecomposition, but only for 2D objects
+    // (see there and in EnhancedCustomShape3d::Create3DObject to read more).
+    // This exception may be removed later when AutoShapes will create primitives directly.
+    // So, currently remove the ShadowAttribute from the ItemSet to not apply it to any
+    // 2D helper shape.
     ClearItem(SDRATTR_SHADOW);
 
     Point aP( pCustomShapeObj->GetSnapRect().Center() );
@@ -731,23 +740,23 @@ EnhancedCustomShape2d::EnhancedCustomShape2d( SdrObject* pAObj ) :
     /*const sal_Int32* pDefData =*/ ApplyShapeAttributes( rGeometryItem );
     switch( eSpType )
     {
-        case mso_sptCan :                       nColorData = 0x20400000; break;

-        case mso_sptCube :                      nColorData = 0x302e0000; break;

-        case mso_sptActionButtonBlank :         nColorData = 0x502ce400; break;

-        case mso_sptActionButtonHome :          nColorData = 0x702ce4ce; break;

-        case mso_sptActionButtonHelp :          nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonInformation :   nColorData = 0x702ce4c5; break;

-        case mso_sptActionButtonBackPrevious :  nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonForwardNext :   nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonBeginning :     nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonEnd :           nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonReturn :        nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonDocument :      nColorData = 0x702ce4ec; break;

-        case mso_sptActionButtonSound :         nColorData = 0x602ce4c0; break;

-        case mso_sptActionButtonMovie :         nColorData = 0x602ce4c0; break;

-        case mso_sptBevel :                     nColorData = 0x502ce400; break;

-        case mso_sptFoldedCorner :              nColorData = 0x20e00000; break;

-        case mso_sptSmileyFace :                nColorData = 0x20e00000; break;

+        case mso_sptCan :                       nColorData = 0x20400000; break;
+        case mso_sptCube :                      nColorData = 0x302e0000; break;
+        case mso_sptActionButtonBlank :         nColorData = 0x502ce400; break;
+        case mso_sptActionButtonHome :          nColorData = 0x702ce4ce; break;
+        case mso_sptActionButtonHelp :          nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonInformation :   nColorData = 0x702ce4c5; break;
+        case mso_sptActionButtonBackPrevious :  nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonForwardNext :   nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonBeginning :     nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonEnd :           nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonReturn :        nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonDocument :      nColorData = 0x702ce4ec; break;
+        case mso_sptActionButtonSound :         nColorData = 0x602ce4c0; break;
+        case mso_sptActionButtonMovie :         nColorData = 0x602ce4c0; break;
+        case mso_sptBevel :                     nColorData = 0x502ce400; break;
+        case mso_sptFoldedCorner :              nColorData = 0x20e00000; break;
+        case mso_sptSmileyFace :                nColorData = 0x20e00000; break;
         case mso_sptNil :
         {
             if( sShapeType.getLength() > 4 &&
@@ -1701,19 +1710,19 @@ void EnhancedCustomShape2d::CreateSubPath( sal_uInt16& rSrcPt, sal_uInt16& rSegm
 
     if(aNewB2DPolyPolygon.count())
     {
-        if( !bLineGeometryNeededOnly )

-        {

-            // hack aNewB2DPolyPolygon to fill logic rect - this is

-            // needed to produce gradient fills that look like mso

-            aNewB2DPolygon.clear();

-            aNewB2DPolygon.append(basegfx::B2DPoint(0,0));

-            aNewB2DPolyPolygon.append(aNewB2DPolygon);

-

-            aNewB2DPolygon.clear();

-            aNewB2DPolygon.append(basegfx::B2DPoint(aLogicRect.GetWidth(),

-                                                    aLogicRect.GetHeight()));

-            aNewB2DPolyPolygon.append(aNewB2DPolygon);

-        }

+        if( !bLineGeometryNeededOnly )
+        {
+            // hack aNewB2DPolyPolygon to fill logic rect - this is
+            // needed to produce gradient fills that look like mso
+            aNewB2DPolygon.clear();
+            aNewB2DPolygon.append(basegfx::B2DPoint(0,0));
+            aNewB2DPolyPolygon.append(aNewB2DPolygon);
+
+            aNewB2DPolygon.clear();
+            aNewB2DPolygon.append(basegfx::B2DPoint(aLogicRect.GetWidth(),
+                                                    aLogicRect.GetHeight()));
+            aNewB2DPolyPolygon.append(aNewB2DPolygon);
+        }
         // #i37011#
         bool bForceCreateTwoObjects(false);