Browse Source

fix review comments from bdahl

Gavin Sharp 13 years ago
parent
commit
117582b2ea
  1. 6
      extensions/firefox/bootstrap.js
  2. 10
      extensions/firefox/components/PdfStreamConverter.js

6
extensions/firefox/bootstrap.js vendored

@ -13,16 +13,16 @@ let Cu = Components.utils; @@ -13,16 +13,16 @@ let Cu = Components.utils;
Cu.import('resource://gre/modules/Services.jsm');
function getBoolPref(pref, default) {
function getBoolPref(pref, def) {
try {
return Services.prefs.getBoolPref(pref);
} catch (ex) {
return default;
return def;
}
}
function setStringPref(pref, value) {
let str = Cc["@mozilla.org/supports-string;1"]
let str = Cc['@mozilla.org/supports-string;1']
.createInstance(Ci.nsISupportsString);
str.data = value;
Services.prefs.setComplexValue(pref, Ci.nsISupportsString, str);

10
extensions/firefox/components/PdfStreamConverter.js

@ -21,26 +21,26 @@ let privateBrowsing = Cc['@mozilla.org/privatebrowsing;1'] @@ -21,26 +21,26 @@ let privateBrowsing = Cc['@mozilla.org/privatebrowsing;1']
.getService(Ci.nsIPrivateBrowsingService);
let inPrivateBrowswing = privateBrowsing.privateBrowsingEnabled;
function getBoolPref(pref, default) {
function getBoolPref(pref, def) {
try {
return Services.prefs.getBoolPref(pref);
} catch (ex) {
return default;
return def;
}
}
function setStringPref(pref, value) {
let str = Cc["@mozilla.org/supports-string;1"]
let str = Cc['@mozilla.org/supports-string;1']
.createInstance(Ci.nsISupportsString);
str.data = value;
Services.prefs.setComplexValue(pref, Ci.nsISupportsString, str);
}
function getStringPref(pref, default) {
function getStringPref(pref, def) {
try {
return Services.prefs.getComplexValue(pref, Ci.nsISupportsString).data;
} catch (ex) {
return default;
return def;
}
}

Loading…
Cancel
Save