mirror of
https://github.com/mozilla/pdf.js.git
synced 2025-04-19 22:58:07 +02:00
[Firefox addon] Temporarily disable the ESLint semi
rule, around the DEFAULT_PREFERENCES
declaration, to prevent mozilla-central test errors
While this doesn't actually fix the underlying issue, it should prevent the ESLint errors and thus make future PDF.js updates easier. Compared to updating (and testing) the preprocessor, this seems like a reasonable workaround given its simplicity.
This commit is contained in:
parent
f299473697
commit
b15de69c52
3 changed files with 9 additions and 3 deletions
4
extensions/firefox/bootstrap.js
vendored
4
extensions/firefox/bootstrap.js
vendored
|
@ -45,11 +45,13 @@ function log(str) {
|
|||
}
|
||||
|
||||
function initializeDefaultPreferences() {
|
||||
/* eslint-disable semi */
|
||||
var DEFAULT_PREFERENCES =
|
||||
//#include ../../web/default_preferences.json
|
||||
//#if false
|
||||
"end of DEFAULT_PREFERENCES";
|
||||
"end of DEFAULT_PREFERENCES"
|
||||
//#endif
|
||||
/* eslint-enable semi */
|
||||
|
||||
var defaultBranch = Services.prefs.getDefaultBranch(EXT_PREFIX + ".");
|
||||
var defaultValue;
|
||||
|
|
|
@ -79,11 +79,13 @@ function isDefaultHandler() {
|
|||
}
|
||||
|
||||
function initializeDefaultPreferences() {
|
||||
/* eslint-disable semi */
|
||||
var DEFAULT_PREFERENCES =
|
||||
//#include ../../../web/default_preferences.json
|
||||
//#if false
|
||||
"end of DEFAULT_PREFERENCES";
|
||||
"end of DEFAULT_PREFERENCES"
|
||||
//#endif
|
||||
/* eslint-enable semi */
|
||||
|
||||
var defaultBranch = Services.prefs.getDefaultBranch(PREF_PREFIX + ".");
|
||||
var defaultValue;
|
||||
|
|
|
@ -33,11 +33,13 @@ XPCOMUtils.defineLazyServiceGetter(Svc, "mime",
|
|||
"@mozilla.org/mime;1",
|
||||
"nsIMIMEService");
|
||||
|
||||
/* eslint-disable semi */
|
||||
var DEFAULT_PREFERENCES =
|
||||
//#include ../../../web/default_preferences.json
|
||||
//#if false
|
||||
"end of DEFAULT_PREFERENCES";
|
||||
"end of DEFAULT_PREFERENCES"
|
||||
//#endif
|
||||
/* eslint-enable semi */
|
||||
|
||||
var PdfjsChromeUtils = {
|
||||
// For security purposes when running remote, we restrict preferences
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue