Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Change test-case issue4914 to an annotations test

Browse Source 
This PDF file (see issue 4914) originally regressed in PR 4318, and was subsequently fixed in PR 4915.

I added the PDF file as a (linked) test-case in PR 6481, in an effort to prevent regressions. Since we at that time didn't have the necessary framework in place, in order to correctly test annotations, this almost regressed *again* in PR https://github.com/mozilla/pdf.js/pull/6672#issuecomment-158689392.

In that PDF file, some of the annotations are both printable and hidden, and should definitely *not* be visible on normal display. Hence this patch, which adds the `annotations` flag to the manifest in order to ensure that those annotations won't be rendered when `intent === 'display'`.
This commit is contained in:
Jonas Jenwald 2015-12-21 15:15:26 +01:00
parent 9f23e115e0
commit ae4ec2e347
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
test/test_manifest.json
View File

@ -498,6 +498,7 @@
"link": true, "link": true,
"lastPage": 1, "lastPage": 1,
"type": "eq", "type": "eq",
"annotations": true,
"about": "PDF with annotations, some of which have the Hidden flag set." "about": "PDF with annotations, some of which have the Hidden flag set."
}, },
{ "id": "issue4665-text", { "id": "issue4665-text",