Merge pull request mozilla#2341 from yurydelendik/issue-2337

brendandahl · brendandahl · commit 68cb9850834e · 2012-11-02T12:54:48.000-07:00
Fixes stream loading for XRefStm
diff --git a/src/obj.js b/src/obj.js
@@ -710,6 +710,7 @@ var XRef = (function XRefClosure() {
         error('invalid first and n parameters for ObjStm stream');
       }
       parser = new Parser(new Lexer(stream), false, this);
+      parser.allowStreams = true;
       var i, entries = [], nums = [];
       // read the object numbers to populate cache
       for (i = 0; i < n; ++i) {
diff --git a/test/pdfs/issue2337.pdf.link b/test/pdfs/issue2337.pdf.link
@@ -0,0 +1 @@
+http://mirrors.ctan.org/info/lshort/english/lshort.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -727,6 +727,14 @@
       "rounds": 1,
       "type": "load"
     },
+    {  "id": "issue2337",
+      "file": "pdfs/issue2337.pdf",
+      "md5": "ea10f4131202b9b8f2a6cb7770d3f185",
+      "rounds": 1,
+      "pageLimit": 1,
+      "link": true,
+      "type": "load"
+    },
     {  "id": "issue2074",
       "file": "pdfs/issue2074.pdf",
       "md5": "5e4ba2241fc35d20e44eb52289a569ab",

Original file line number	Diff line number	Diff line change
`@@ -710,6 +710,7 @@ var XRef = (function XRefClosure() {`
`710`	`710`	`error('invalid first and n parameters for ObjStm stream');`
`711`	`711`	`}`
`712`	`712`	`parser = new Parser(new Lexer(stream), false, this);`
	`713`	`+ parser.allowStreams = true;`
`713`	`714`	`var i, entries = [], nums = [];`
`714`	`715`	`// read the object numbers to populate cache`
`715`	`716`	`for (i = 0; i < n; ++i) {`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+http://mirrors.ctan.org/info/lshort/english/lshort.pdf`