tdf#96993: remove obsolete vertical mirror hack from PDF import

Since commit ae0e830f, mirroring
is handled correctly in LO, so no need to handle it specially in
PDF import code. This obsolete handling causes double-mirroring.

Change-Id: Ia873f0426e444cafd80b24b7d209e90eb481d8f4
Reviewed-on: https://gerrit.libreoffice.org/21870Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>

sdext/source/pdfimport/tree/drawtreevisiting.cxx

@@ -215,9 +215,9 @@ void DrawXmlEmitter::fillFrameProps( DrawElement&       rElem,
         if (rElem.MirrorVertical)
         {
             basegfx::B2DHomMatrix mat2;
-            mat2.translate(-0.5, -0.5);
-            mat2.scale(-1, -1);
-            mat2.translate(0.5, 0.5);
+            mat2.translate(0, -0.5);
+            mat2.scale(1, -1);
+            mat2.translate(0, 0.5);
             mat = mat * mat2;
         }
 
@@ -949,13 +949,6 @@ void DrawXmlFinalizer::visit( FrameElement& elem, const std::list< Element* >::c
     aGCProps[ "fo:padding-right" ]               = "0cm";
     aGCProps[ "fo:padding-bottom" ]              = "0cm";
 
-    // remark: vertical mirroring is done in current OOO by
-    // mirroring horzontally and rotating 180 degrees
-    // this is quaint, but unfortunately it seems
-    // mirror=vertical is defined but not implemented in current code
-    if( elem.MirrorVertical )
-        aGCProps[ "style:mirror" ] = "horizontal";
-
     StyleContainer::Style style1( "style:style", props1 );
     StyleContainer::Style subStyle1( "style:graphic-properties", aGCProps );
     style1.SubStyles.push_back(&subStyle1);

sdext/source/pdfimport/tree/pdfiprocessor.cxx

@@ -350,7 +350,7 @@ void PDFIProcessor::setupImage(ImageId nImage)
     pFrame->h = pImageElement->h = aScale.getY();
     pFrame->ZOrder = m_nNextZOrder++;
 
-    if (aScale.getY() > 0)
+    if (aScale.getY() < 0)
         pFrame->MirrorVertical = pImageElement->MirrorVertical = true;
 }