SVG: Use actual closePath operator in PDF syntax

Update cmp files

DEVSIX-2718 also solves DEVSIX-2583

Author: Benoit Lagae

svg/src/main/java/com/itextpdf/svg/renderers/impl/AbstractSvgNodeRenderer.java

@@ -252,7 +252,6 @@ void postDraw(SvgDrawContext context) {
                 } else if (doStroke) {
                     currentCanvas.stroke();
                 }
-                currentCanvas.closePath(); // TODO: see if this is necessary DEVSIX-2583
             }
         }
     }

svg/src/main/java/com/itextpdf/svg/renderers/path/impl/ClosePath.java

@@ -42,6 +42,8 @@ This file is part of the iText (R) project.
  */
 package com.itextpdf.svg.renderers.path.impl;
 
+import com.itextpdf.kernel.pdf.canvas.PdfCanvas;
+
 /***
  * Implements closePath(Z) attribute of SVG's path element
  * */
@@ -56,4 +58,8 @@ public ClosePath(boolean relative) {
         super(relative);
     }
 
+    @Override
+    public void draw(PdfCanvas canvas) {
+        canvas.closePath();
+    }
 }

svg/src/test/java/com/itextpdf/svg/renderers/TransformationApplicationTest.java

@@ -63,7 +63,7 @@ public class TransformationApplicationTest {
 
     @Test
     public void normalDrawTest() {
-        byte[] expected = "1 0 0 1 7.5 0 cm\n0 0 0 rg\nf\nh\n".getBytes(StandardCharsets.UTF_8);
+        byte[] expected = "1 0 0 1 7.5 0 cm\n0 0 0 rg\nf\n".getBytes(StandardCharsets.UTF_8);
 
         ISvgNodeRenderer nodeRenderer = new AbstractSvgNodeRenderer() {
 

svg/src/test/java/com/itextpdf/svg/renderers/impl/ClipPathSvgNodeRendererLowLevelIntegrationTest.java

@@ -172,7 +172,6 @@ public void testAppliedClipPathRenderer() {
                 "-63 -34.79 -34.79 -63 0 -63 c\n" +
                 "34.79 -63 63 -34.79 63 0 c\n" +
                 "f\n" +
-                "h\n" +
                 "Q\n";
         Assert.assertEquals(expected, new String(cv.getContentStream().getBytes()));
     }
@@ -211,7 +210,6 @@ public void testAppliedGroupClipPathRenderer() {
                 "-63 -34.79 -34.79 -63 0 -63 c\n" +
                 "34.79 -63 63 -34.79 63 0 c\n" +
                 "f\n" +
-                "h\n" +
                 "Q\n" +
                 "Q\n";
         Assert.assertEquals(expected, new String(cv.getContentStream().getBytes()));
@@ -258,7 +256,6 @@ public void testEoAppliedGroupClipPathRenderer() {
                 "-63 -34.79 -34.79 -63 0 -63 c\n" +
                 "34.79 -63 63 -34.79 63 0 c\n" +
                 "f\n" +
-                "h\n" +
                 "Q\n" +
                 "q\n" +
                 "% rect\n" +
@@ -272,7 +269,6 @@ public void testEoAppliedGroupClipPathRenderer() {
                 "-63 -34.79 -34.79 -63 0 -63 c\n" +
                 "34.79 -63 63 -34.79 63 0 c\n" +
                 "f\n" +
-                "h\n" +
                 "Q\n" +
                 "Q\n";
         Assert.assertEquals(expected, new String(cv.getContentStream().getBytes()));