SVG: Revert previous commits for tdf#95332 and tdf#95733

I was wrong for thinking that there was a problem when
elements inside def elements were not written.
Actually, Cristina's commit
( 5e27d23d ) was correct
as elements inside defs don't have to be written at least
they're refered by <use> element, which isn't supported in Libo
right now. Working on implementing support for use element

Change-Id: Id0b8d8e9b27887f75cc961881daeb82089b2e62b
Reviewed-on: https://gerrit.libreoffice.org/20333Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>

filter/source/svg/svgreader.cxx

@@ -88,22 +88,20 @@ template<typename Func> void visitElements(Func& rFunc,
     // notify children processing
     rFunc.push();
 
+    if (eCaller == SHAPE_WRITER && rElem->getTagName() == "defs")
+        return;
+
     // recurse over children
     uno::Reference<xml::dom::XNodeList> xChildren( rElem->getChildNodes() );
     const sal_Int32 nNumNodes( xChildren->getLength() );
     for( sal_Int32 i=0; i<nNumNodes; ++i )
     {
         if( xChildren->item(i)->getNodeType() == xml::dom::NodeType_ELEMENT_NODE ){
-            //tdf#65864
-            //TODO: support clipPath
-            if( !(eCaller == SHAPE_WRITER &&
-                rElem->getTagName() == "clipPath")){
-                    visitElements( rFunc,
-                                   uno::Reference<xml::dom::XElement>(
-                                       xChildren->item(i),
-                                       uno::UNO_QUERY_THROW),
-                                   eCaller );
-            }
+            visitElements( rFunc,
+                uno::Reference<xml::dom::XElement>(
+                xChildren->item(i),
+                uno::UNO_QUERY_THROW),
+                eCaller );
         }
     }