Merge pull request #18726 from calixteman/issue18058

Consume any pending path before drawing an annotation
mozilla · Sep 10, 2024 · 0ac7f29 · 0ac7f29
2 parents 66ed4b9 + 518b776
commit 0ac7f29
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/src/display/canvas.js b/src/display/canvas.js
@@ -2701,9 +2701,12 @@ class CanvasGraphics {
       } else {
         resetCtxToDefault(this.ctx);
 
+        // Consume a potential path before clipping.
+        this.endPath();
+
         this.ctx.rect(rect[0], rect[1], width, height);
         this.ctx.clip();
-        this.endPath();
+        this.ctx.beginPath();
       }
     }
 

diff --git a/test/pdfs/issue18058.pdf.link b/test/pdfs/issue18058.pdf.link
@@ -0,0 +1 @@
+https://github.com/mozilla/pdf.js/files/15269193/file.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -10474,5 +10474,13 @@
         "pageIndex": 0
       }
     }
+  },
+  {
+    "id": "issue18058",
+    "file": "pdfs/issue18058.pdf",
+    "md5": "da8fcdf25c83ecb2d4be315be5affc3c",
+    "rounds": 1,
+    "link": true,
+    "type": "eq"
   }
 ]
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		https://github.com/mozilla/pdf.js/files/15269193/file.pdf