SVG: don't draw rect with negative or zero height or width

DEVSIX-8760

Autoported commit.
Original commit hash: [457ddde85]

Author: AnhelinaM

itext.tests/itext.svg.tests/itext/svg/renderers/FillTest.cs

@@ -121,7 +121,6 @@ public virtual void InvalidUrlFillTest() {
         [NUnit.Framework.Test]
         [LogMessage(SvgLogMessageConstant.UNMAPPED_TAG, Count = 4)]
         public virtual void TextFillFallbackTest() {
-            //TODO update cmp file after DEVSIX-2915 will be fixed
             ConvertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER, "textFillFallbackTest");
         }
 

itext.tests/itext.svg.tests/itext/svg/renderers/StrokeTest.cs

@@ -102,8 +102,12 @@ public virtual void StrokeAttributesTest() {
 
         [NUnit.Framework.Test]
         public virtual void HeightWidthZeroTest() {
-            //TODO DEVSIX-8760 : change cmp after fix
-            ConvertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER, "height-width-zero");
+            ConvertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER, "heightWidthZero");
+        }
+
+        [NUnit.Framework.Test]
+        public virtual void HeightWidthNegativeTest() {
+            ConvertAndCompareSinglePage(SOURCE_FOLDER, DESTINATION_FOLDER, "heightWidthNegative");
         }
 
         [NUnit.Framework.Test]

itext.tests/itext.svg.tests/itext/svg/renderers/impl/ClipPathSvgNodeRendererLowLevelIntegrationTest.cs

@@ -78,7 +78,7 @@ public virtual void TestRectClipPathRenderer() {
             clipRenderer.AddChild(rectRenderer);
             clipRenderer.SetClippedRenderer(new RectangleSvgNodeRenderer());
             clipRenderer.Draw(sdc);
-            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", iText.Commons.Utils.JavaUtil.GetStringForBytes
+            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\nf\nQ\n", iText.Commons.Utils.JavaUtil.GetStringForBytes
                 (cv.GetContentStream().GetBytes()));
         }
 
@@ -93,7 +93,7 @@ public virtual void TestRectClipPathEoRendererNoChange() {
             clipRenderer.AddChild(rectRenderer);
             clipRenderer.SetClippedRenderer(new RectangleSvgNodeRenderer());
             clipRenderer.Draw(sdc);
-            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", iText.Commons.Utils.JavaUtil.GetStringForBytes
+            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW\nn\n0 0 0 rg\n% rect\nf\nQ\n", iText.Commons.Utils.JavaUtil.GetStringForBytes
                 (cv.GetContentStream().GetBytes()));
         }
 
@@ -108,8 +108,8 @@ public virtual void TestRectEoClipPathRenderer() {
             clipRenderer.AddChild(rectRenderer);
             clipRenderer.SetClippedRenderer(new RectangleSvgNodeRenderer());
             clipRenderer.Draw(sdc);
-            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW*\nn\n0 0 0 rg\n% rect\n0 0 0 0 re\nf\nQ\n", 
-                iText.Commons.Utils.JavaUtil.GetStringForBytes(cv.GetContentStream().GetBytes()));
+            NUnit.Framework.Assert.AreEqual("q\n% rect\n0 0 300 300 re\nW*\nn\n0 0 0 rg\n% rect\nf\nQ\n", iText.Commons.Utils.JavaUtil.GetStringForBytes
+                (cv.GetContentStream().GetBytes()));
         }
 
         [NUnit.Framework.Test]

itext.tests/itext.svg.tests/resources/itext/svg/renderers/impl/RectangleSvgNodeRendererTest/cmp_rectangleNoWidthNoHeight.pdf

@@ -58,6 +58,9 @@ protected internal override void DoDraw(SvgDrawContext context) {
             PdfCanvas cv = context.GetCurrentCanvas();
             cv.WriteLiteral("% rect\n");
             SetParameters(context);
+            if (width <= 0 || height <= 0) {
+                return;
+            }
             bool singleValuePresent = (rxPresent && !ryPresent) || (!rxPresent && ryPresent);
             AffineTransform transform = ApplyNonScalingStrokeTransform(context);
             if (!rxPresent && !ryPresent) {