Skip to content

Commit

Permalink
Change test-case issue4914 to an annotations test
Browse files Browse the repository at this point in the history
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 mozilla#6672 (comment).

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'`.
  • Loading branch information
Snuffleupagus authored and iamkhush committed Dec 26, 2015
1 parent 98526fb commit ddc14a3
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions test/test_manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -498,6 +498,7 @@
"link": true,
"lastPage": 1,
"type": "eq",
"annotations": true,
"about": "PDF with annotations, some of which have the Hidden flag set."
},
{ "id": "issue4665-text",
Expand Down

0 comments on commit ddc14a3

Please sign in to comment.