Merge pull request #18974 from Snuffleupagus/issue-18973

Allow `StreamsSequenceStream` to skip sub-streams that are not actual Streams (issue 18973)
mozilla · Oct 29, 2024 · 3a85479 · 3a85479
2 parents 93961e2 + 48a1858
commit 3a85479
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/src/core/decode_stream.js b/src/core/decode_stream.js
@@ -132,6 +132,8 @@ class DecodeStream extends BaseStream {
 
 class StreamsSequenceStream extends DecodeStream {
   constructor(streams, onError = null) {
+    streams = streams.filter(s => s instanceof BaseStream);
+
     let maybeLength = 0;
     for (const stream of streams) {
       maybeLength +=

diff --git a/test/pdfs/issue18973.pdf.link b/test/pdfs/issue18973.pdf.link
@@ -0,0 +1 @@
+https://github.com/user-attachments/files/17550758/doc1520828609.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -2057,6 +2057,15 @@
     "firstPage": 3,
     "type": "eq"
   },
+  {
+    "id": "issue18973",
+    "file": "pdfs/issue18973.pdf",
+    "md5": "b9bbd312269862bf39bb2a31023a9d02",
+    "link": true,
+    "rounds": 1,
+    "firstPage": 45,
+    "type": "eq"
+  },
   {
     "id": "issue9262",
     "file": "pdfs/issue9262_reduced.pdf",
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		https://github.com/user-attachments/files/17550758/doc1520828609.pdf