Commit 83dac765 authored by Pier Angelo Vendrame's avatar Pier Angelo Vendrame 🎃
Browse files

fixup! BB 41581: Hide NoScript extension's toolbar button by default. 

Fix lints.
parent 45d420a0

toolkit/mozapps/extensions/content/components/addon-card.mjs

+2 −0
Original line number Diff line number Diff line
@@ -45,6 +45,8 @@ const PRIVATE_BROWSING_PERMS = { 
  origins: [],

};



const HIDE_NO_SCRIPT_PREF = "extensions.hideNoScript";



/**

 * A card component for managing an add-on. It should be initialized by setting

 * the add-on with `setAddon()` before being connected to the document.

toolkit/mozapps/extensions/content/components/addon-details.mjs

+4 −1
Original line number Diff line number Diff line
@@ -22,6 +22,9 @@ const { XPCOMUtils } = ChromeUtils.importESModule( 
  "resource://gre/modules/XPCOMUtils.sys.mjs"

);

const lazy = {};

ChromeUtils.defineESModuleGetters(lazy, {

  ExtensionCommon: "resource://gre/modules/ExtensionCommon.sys.mjs",

});

XPCOMUtils.defineLazyPreferenceGetter(

  lazy,

  "DATA_COLLECTION_PERMISSIONS_ENABLED",
@@ -587,7 +590,7 @@ export class AddonDetails extends AboutAddonsHTMLElement { 
      const visibilityLabel = visibilityRow.querySelector(

        ".addon-noscript-visibility-label"

      );

      visibilityLabel.id = ExtensionCommon.makeWidgetId(

      visibilityLabel.id = lazy.ExtensionCommon.makeWidgetId(

        `${addon.id}-noscript-visibility-label`

      );

      visibilityRow.setAttribute("aria-labelledby", visibilityLabel.id);