Bug 1405166 - Remove system-disabled pref from Screenshots; r?kmag draft
authorJared Hirsch <ohai@6a68.net>
Tue, 03 Oct 2017 09:09:38 -0700
changeset 674334 d9d4a0f1039717af26f81dfbfe0b17ad0c04ed8d
parent 673986 65dac33a5682f3ec5a675e7f3314b0c1520a13fa
child 674540 7ed176f744c5a30deef44412f0c0d381827a9b71
child 675712 a98cde4b7eb87f72968d17603fb18fa9502c151e
push id82797
push userbmo:jhirsch@mozilla.com
push dateTue, 03 Oct 2017 16:10:09 +0000
reviewerskmag
bugs1405166
milestone58.0a1
Bug 1405166 - Remove system-disabled pref from Screenshots; r?kmag MozReview-Commit-ID: KuDfW6EZMWZ
browser/extensions/screenshots/bootstrap.js
--- a/browser/extensions/screenshots/bootstrap.js
+++ b/browser/extensions/screenshots/bootstrap.js
@@ -2,17 +2,16 @@
 // TODO: re-enable
 /* eslint-disable */
 const OLD_ADDON_PREF_NAME = "extensions.jid1-NeEaf3sAHdKHPA@jetpack.deviceIdInfo";
 const OLD_ADDON_ID = "jid1-NeEaf3sAHdKHPA@jetpack";
 const ADDON_ID = "screenshots@mozilla.org";
 const TELEMETRY_ENABLED_PREF = "datareporting.healthreport.uploadEnabled";
 const PREF_BRANCH = "extensions.screenshots.";
 const USER_DISABLE_PREF = "extensions.screenshots.disabled";
-const SYSTEM_DISABLE_PREF = "extensions.screenshots.system-disabled";
 
 const { interfaces: Ci, utils: Cu } = Components;
 Cu.import("resource://gre/modules/XPCOMUtils.jsm");
 XPCOMUtils.defineLazyModuleGetter(this, "AddonManager",
                                   "resource://gre/modules/AddonManager.jsm");
 XPCOMUtils.defineLazyModuleGetter(this, "AppConstants",
                                   "resource://gre/modules/AppConstants.jsm");
 XPCOMUtils.defineLazyModuleGetter(this, "Console",
@@ -40,17 +39,17 @@ const prefObserver = {
 
   unregister() {
     prefs.removeObserver(PREF_BRANCH, this, false); // eslint-disable-line mozilla/no-useless-parameters
   },
 
   observe(aSubject, aTopic, aData) {
     // aSubject is the nsIPrefBranch we're observing (after appropriate QI)
     // aData is the name of the pref that's been changed (relative to aSubject)
-    if (aData == USER_DISABLE_PREF || aData == SYSTEM_DISABLE_PREF) {
+    if (aData == USER_DISABLE_PREF) {
       // eslint-disable-next-line promise/catch-or-return
       appStartupPromise = appStartupPromise.then(handleStartup);
     }
   }
 };
 
 const appStartupObserver = {
   register() {
@@ -158,17 +157,17 @@ function install(data, reason) {} // esl
 
 function uninstall(data, reason) {} // eslint-disable-line no-unused-vars
 
 function getBoolPref(pref) {
   return prefs.getPrefType(pref) && prefs.getBoolPref(pref);
 }
 
 function shouldDisable() {
-  return getBoolPref(USER_DISABLE_PREF) || getBoolPref(SYSTEM_DISABLE_PREF);
+  return getBoolPref(USER_DISABLE_PREF);
 }
 
 function handleStartup() {
   const webExtension = LegacyExtensionsUtils.getEmbeddedExtensionFor({
     id: ADDON_ID,
     resourceURI: addonResourceURI
   });