1652145 - Add pref to enable/disable form support in PDF.js

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, enhancement)

Description

[Brendan Dahl [:bdahl]]

For implementing this there are already a few firefox prefs that are read by pdf.js e.g. enablePrintAutoRotate

Comment 1

[Blake Winton (:bwinton) (:☕️)]

I believe we're going to use pdfjs.renderInteractiveForms for the name of the pref…

Comment 2

[Brendan Dahl [:bdahl]]

Attachment: Bug 1652145 - Only enable PDF forms for early beta or earlier. r=bgrins

To keep PDFJS's mechanism to setup default prefs from overriding values
defined in all.js, I had to add a check to see if the pref is already
defined.

Comment 3

[Brendan Dahl [:bdahl]]

Attachment: Bug 1652145 - Add pdfjs forms smoke test. r=bgrins

• ensures pref initialization is working correctly
• ensures form element is there when forms are enabled
• ensures forms can be disabled

Depends on D87009

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51953e2f01d7
Only enable PDF forms for early beta or earlier. r=bgrins
https://hg.mozilla.org/integration/autoland/rev/49dd6f42bc7e
Add pdfjs forms smoke test. r=bgrins

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/51953e2f01d7
https://hg.mozilla.org/mozilla-central/rev/49dd6f42bc7e