Merge pull request #19715 from calixteman/issue18529

calixteman · web-flow · commit bcba053a5e7e · 2025-03-24T14:41:29.000+01:00
Take into account the path and the line width when consuming a stroked path
diff --git a/src/display/canvas.js b/src/display/canvas.js
@@ -1664,7 +1664,13 @@ class CanvasGraphics {
       }
     }
     if (consumePath) {
-      this.consumePath(path, this.current.getClippedPathBoundingBox());
+      this.consumePath(
+        path,
+        this.current.getClippedPathBoundingBox(
+          PathType.STROKE,
+          getCurrentTransform(this.ctx)
+        )
+      );
     }
     // Restore the global alpha to the fill alpha
     ctx.globalAlpha = this.current.fillAlpha;
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
@@ -715,3 +715,4 @@
 !red_stamp.pdf
 !issue19633.pdf
 !issue19424.pdf
+!issue18529.pdf
diff --git a/test/pdfs/issue18529.pdf b/test/pdfs/issue18529.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -12012,5 +12012,12 @@
     "firstPage": 1,
     "lastPage": 1,
     "type": "eq"
+  },
+  {
+    "id": "issue18529",
+    "file": "pdfs/issue18529.pdf",
+    "md5": "f95d2cbdd904f5acf922e088cc3cb153",
+    "rounds": 1,
+    "type": "eq"
   }
 ]

Original file line number	Diff line number	Diff line change
`@@ -1664,7 +1664,13 @@ class CanvasGraphics {`
`1664`	`1664`	`}`
`1665`	`1665`	`}`
`1666`	`1666`	`if (consumePath) {`
`1667`		`- this.consumePath(path, this.current.getClippedPathBoundingBox());`
	`1667`	`+ this.consumePath(`
	`1668`	`+ path,`
	`1669`	`+ this.current.getClippedPathBoundingBox(`
	`1670`	`+ PathType.STROKE,`
	`1671`	`+ getCurrentTransform(this.ctx)`
	`1672`	`+ )`
	`1673`	`+ );`
`1668`	`1674`	`}`
`1669`	`1675`	`// Restore the global alpha to the fill alpha`
`1670`	`1676`	`ctx.globalAlpha = this.current.fillAlpha;`