Closed Bug 1330196 Opened 5 years ago Closed 5 years ago

[jsplugins] Can not render pdf with latest PDFium binary

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 54
Tracking Status
firefox54 --- fixed

People

(Reporter: lochang, Assigned: lochang)

References

Details

Attachments

(1 file)

We can not normally render the pdf with the latest PDFium binary. The page is blank but UI is still there. We should dig into the problem see if there is anything we can fix in runtime.
Assignee: nobody → lochang
We can not normally render the pdf with the PDFium binary which is built after 2016, Nov 15.
Hi Bruce,

Would you please review the patch? The patch fixes the rendering problem with latest PDFium binary.

Recently, Chromium improves pinch-zoom for PDF [1]. PostMessage of type viewport takes an addition parameter pinchPhase. We workaround here by adding a pinchPhase of value 0 to make sure that viewing pdf works normally. 

[1] Chromium revision: 6e1abbfb2450eedddb1ab128be1b31cc93104e41
Attachment #8835446 - Flags: review?(brsun)
Comment on attachment 8835446 [details] [diff] [review]
Bug 1330196 - Can not render pdf with latest PDFium binary, r=brsun

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

LGTM
Attachment #8835446 - Flags: review?(brsun) → review+
Keywords: checkin-needed
Attachment #8835446 - Attachment description: Bug 1330196 - Can not render pdf with latest PDFium binary → Bug 1330196 - Can not render pdf with latest PDFium binary, r=brsun
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/80e155870904
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
You need to log in before you can comment on or make changes to this bug.