remove unnecessary method

the call to GetObjectItemSet() in
OleProperties::ForceDefaultAttributes() is unnecessary, because
it is always called before ForceDefaultAttributes is called, and
having removed that, the rest of the method becomes unnecessary too.

Change-Id: Ibb09020837d47cb9eb1fb8b5deca1defc1465108
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151237
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

svx/inc/sdr/properties/oleproperties.hxx

@@ -42,10 +42,6 @@ namespace sdr::properties
 
             // Clone() operator, normally just calls the local copy constructor
             virtual std::unique_ptr<BaseProperties> Clone(SdrObject& rObj) const override;
-
-            // force default attributes for a specific object type, called from
-            // DefaultProperties::GetObjectItemSet() if a new ItemSet is created
-            virtual void ForceDefaultAttributes() override;
         };
 } // end of namespace sdr::properties
 

svx/source/sdr/properties/oleproperties.cxx

@@ -60,15 +60,6 @@ namespace sdr::properties
         {
             return std::unique_ptr<BaseProperties>(new OleProperties(*this, rObj));
         }
-
-        void OleProperties::ForceDefaultAttributes()
-        {
-            // call parent
-            RectangleProperties::ForceDefaultAttributes();
-
-            // force ItemSet
-            GetObjectItemSet();
-        }
 } // end of namespace
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */