Fix tdf#87509 - default sdr attribute is special object.

The drawinglayer attributes signal defaultness by object identity, not
value equalness. That should fix a number of subtle cache and redraw
problems.

Change-Id: I049ffda228a48db71cef108571805c6e41e4b149

drawinglayer/source/attribute/fillgradientattribute.cxx

@@ -142,6 +142,10 @@ namespace drawinglayer
 
         bool FillGradientAttribute::operator==(const FillGradientAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpFillGradientAttribute == mpFillGradientAttribute;
         }
 

drawinglayer/source/attribute/fillgraphicattribute.cxx

@@ -120,6 +120,10 @@ namespace drawinglayer
 
         bool FillGraphicAttribute::operator==(const FillGraphicAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpFillGraphicAttribute == mpFillGraphicAttribute;
         }
 

drawinglayer/source/attribute/fillhatchattribute.cxx

@@ -131,6 +131,10 @@ namespace drawinglayer
 
         bool FillHatchAttribute::operator==(const FillHatchAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpFillHatchAttribute == mpFillHatchAttribute;
         }
 

drawinglayer/source/attribute/lineattribute.cxx

@@ -118,6 +118,10 @@ namespace drawinglayer
 
         bool LineAttribute::operator==(const LineAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpLineAttribute == mpLineAttribute;
         }
 

drawinglayer/source/attribute/linestartendattribute.cxx

@@ -110,6 +110,10 @@ namespace drawinglayer
 
         bool LineStartEndAttribute::operator==(const LineStartEndAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpLineStartEndAttribute == mpLineStartEndAttribute;
         }
 

drawinglayer/source/attribute/sdrfillattribute.cxx

@@ -126,6 +126,10 @@ namespace drawinglayer
 
         bool SdrFillAttribute::operator==(const SdrFillAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrFillAttribute == mpSdrFillAttribute;
         }
 

drawinglayer/source/attribute/sdrfillgraphicattribute.cxx

@@ -161,6 +161,10 @@ namespace drawinglayer
 
         bool SdrFillGraphicAttribute::operator==(const SdrFillGraphicAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrFillGraphicAttribute == mpSdrFillGraphicAttribute;
         }
 

drawinglayer/source/attribute/sdrlightingattribute3d.cxx

@@ -102,6 +102,10 @@ namespace drawinglayer
 
         bool SdrLightingAttribute::operator==(const SdrLightingAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrLightingAttribute == mpSdrLightingAttribute;
         }
 

drawinglayer/source/attribute/sdrlineattribute.cxx

@@ -142,6 +142,10 @@ namespace drawinglayer
 
         bool SdrLineAttribute::operator==(const SdrLineAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrLineAttribute == mpSdrLineAttribute;
         }
 

drawinglayer/source/attribute/sdrlinestartendattribute.cxx

@@ -144,6 +144,10 @@ namespace drawinglayer
 
         bool SdrLineStartEndAttribute::operator==(const SdrLineStartEndAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrLineStartEndAttribute == mpSdrLineStartEndAttribute;
         }
 

drawinglayer/source/attribute/sdrsceneattribute3d.cxx

@@ -123,6 +123,10 @@ namespace drawinglayer
 
         bool SdrSceneAttribute::operator==(const SdrSceneAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpSdrSceneAttribute == mpSdrSceneAttribute;
         }
 

drawinglayer/source/attribute/sdrshadowattribute.cxx

@@ -109,6 +109,10 @@ namespace drawinglayer
 
         bool SdrShadowAttribute::operator==(const SdrShadowAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return mpSdrShadowAttribute == rCandidate.mpSdrShadowAttribute;
         }
 

drawinglayer/source/attribute/strokeattribute.cxx

@@ -110,6 +110,10 @@ namespace drawinglayer
 
         bool StrokeAttribute::operator==(const StrokeAttribute& rCandidate) const
         {
+            // tdf#87509 default attr is always != non-default attr, even with same values
+            if(rCandidate.isDefault() != isDefault())
+                return false;
+
             return rCandidate.mpStrokeAttribute == mpStrokeAttribute;
         }