Add rendering test for PDF.js

NEW
Unassigned

Status

()

Firefox
PDF Viewer
5 years ago
3 years ago

People

(Reporter: Mack Duan, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [pdfjs-c-testing])

Attachments

(1 attachment, 2 obsolete attachments)

396.01 KB, patch
bdahl
: review-
Details | Diff | Splinter Review
Comment hidden (empty)
(Reporter)

Comment 1

5 years ago
Created attachment 739324 [details] [diff] [review]
Patch file
Attachment #739324 - Flags: review?(bdahl)
(Reporter)

Comment 2

5 years ago
Try run: https://tbpl.mozilla.org/?tree=Try&rev=5cfd4cc28845
(Reporter)

Comment 3

5 years ago
Created attachment 739694 [details]
Patch file v2

Adds a forgotten line to Makefile.in
Attachment #739324 - Attachment is obsolete: true
Attachment #739324 - Flags: review?(bdahl)
Attachment #739694 - Flags: review?(bdahl)
(Reporter)

Comment 4

5 years ago
Created attachment 739697 [details] [diff] [review]
Patch file v3
Attachment #739694 - Attachment is obsolete: true
Attachment #739694 - Flags: review?(bdahl)
Attachment #739697 - Flags: review?(bdahl)
(Reporter)

Comment 5

5 years ago
These tests will require https://github.com/mozilla/pdf.js/pull/2914 to be uplifted since it requires the custom 'pagerender' event.

Updated

5 years ago
Whiteboard: [pdfjs-c-testing]
Comment on attachment 739697 [details] [diff] [review]
Patch file v3

Review of attachment 739697 [details] [diff] [review]:
-----------------------------------------------------------------

Clearing this old review. If someone wants to update this patch and take over work I'll review.
Attachment #739697 - Flags: review?(bdahl) → review-
You need to log in before you can comment on or make changes to this bug.