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

Merge pull request #15459 from calixteman/1774427

Browse Source 
Don't use window.print in the Firefox builtin viewer (bug 1774427)
This commit is contained in:
calixteman 2022-09-19 15:07:10 +02:00 committed by GitHub
commit bf78ccac30
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
web/app.js
View File

@ -152,6 +152,10 @@ class DefaultExternalServices {
static reportTelemetry(data) {} static reportTelemetry(data) {}
static print() {
window.print();
}
static createDownloadManager(options) { static createDownloadManager(options) {
throw new Error("Not implemented: createDownloadManager"); throw new Error("Not implemented: createDownloadManager");
} }
@ -1897,7 +1901,7 @@ const PDFViewerApplication = {
if (!this.supportsPrinting) { if (!this.supportsPrinting) {
return; return;
} }
window.print(); this.externalServices.print();
}, },
bindEvents() { bindEvents() {

4
web/firefoxcom.js
View File

@ -402,6 +402,10 @@ class FirefoxExternalServices extends DefaultExternalServices {
FirefoxCom.request("updateEditorStates", data); FirefoxCom.request("updateEditorStates", data);
} }
static print() {
FirefoxCom.request("print", null);
}
static createL10n(options) { static createL10n(options) {
const mozL10n = document.mozL10n; const mozL10n = document.mozL10n;
// TODO refactor mozL10n.setExternalLocalizerServices // TODO refactor mozL10n.setExternalLocalizerServices