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

[CRX] Fix error that broke the Chrome extension

Browse Source 
PR #9493 moved from `appConfig.defaultUrl` to `AppOptions.get('defaultUrl')`.
However, it forgot to replace `appConfig.defaultUrl` in chromecom.js,
and as a result the extension is not able to open any PDF file.

This patch fixes that issue.
This commit is contained in:
Rob Wu 2018-03-08 18:44:12 +01:00
parent c33bf800cc
commit 39e0b4b589
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
web/chromecom.js
View File

@ -15,6 +15,7 @@
/* globals chrome */
import { DefaultExternalServices, PDFViewerApplication } from './app';
import { AppOptions } from './app_options';
import { BasePreferences } from './preferences';
import { DownloadManager } from './download_manager';
import { GenericL10n } from './genericl10n';
@ -357,8 +358,9 @@ class ChromePreferences extends BasePreferences {
let ChromeExternalServices = Object.create(DefaultExternalServices);
ChromeExternalServices.initPassiveLoading = function(callbacks) {
let { appConfig, overlayManager, } = PDFViewerApplication;
ChromeCom.resolvePDFFile(appConfig.defaultUrl, overlayManager,
let { overlayManager, } = PDFViewerApplication;
// defaultUrl is set in viewer.js
ChromeCom.resolvePDFFile(AppOptions.get('defaultUrl'), overlayManager,
function(url, length, originalURL) {
callbacks.onOpenWithURL(url, length, originalURL);
});