Commit acecadf9 authored by Arthur Edelstein's avatar Arthur Edelstein Committed by Pier Angelo Vendrame
Browse files

BB 18905: Hide unwanted items from help menu 

Bug 25660: Remove the "New Private Window" option
parent ace945b0

browser/base/content/browser-menubar.inc

+4 −0
Original line number Diff line number Diff line
@@ -423,6 +423,7 @@ 
<!-- Note: Items under here are cloned to the AppMenu Help submenu. The cloned items

     have their strings defined by appmenu-data-l10n-id. -->

                <menuitem id="menu_openHelp"

                          hidden="true"

                          data-l10n-id="menu-get-help"

                          appmenu-data-l10n-id="appmenu-get-help"

                          />
@@ -432,9 +433,11 @@ 
                          disabled="true"

                          appmenu-data-l10n-id="appmenuitem-report-broken-site"/>

                <menuitem id="feedbackPage"

                          hidden="true"

                          data-l10n-id="menu-help-share-ideas"

                          appmenu-data-l10n-id="appmenu-help-share-ideas"/>

                <menuitem id="helpSafeMode"

                          hidden="true"

                          data-l10n-id="menu-help-enter-troubleshoot-mode2"

                          appmenu-data-l10n-id="appmenu-help-enter-troubleshoot-mode2"/>

                <menuitem id="troubleShooting"
@@ -451,6 +454,7 @@ 
                          appmenu-data-l10n-id="appmenu-help-not-deceptive"

                          hidden="true"/>

                <menuitem id="helpSwitchDevice"

                          hidden="true"

                          data-l10n-id="menu-help-switch-device"

                          appmenu-data-l10n-id="appmenu-help-switch-device"/>

                <menuseparator id="aboutSeparator"/>

browser/base/content/browser-safebrowsing.js

+7 −0
Original line number Diff line number Diff line
@@ -7,6 +7,13 @@ 


var gSafeBrowsing = {

  setReportPhishingMenu() {

    // tor-browser#18905: hide these menu entries

    if (

      !Services.prefs.getBoolPref("browser.safebrowsing.phishing.enabled", true)

    ) {

      return;

    }



    // In order to detect whether or not we're at the phishing warning

    // page, we have to check the documentURI instead of the currentURI.

    // This is because when the DocShell loads an error page, the

browser/components/reportbrokensite/ReportBrokenSite.sys.mjs

+2 −3
Original line number Diff line number Diff line
@@ -455,9 +455,8 @@ export var ReportBrokenSite = new (class ReportBrokenSite { 
    // Altering the disabled attribute on the command does not propagate

    // the change to the related menuitems (see bug 805653), so we change them all.

    const cmd = document.getElementById("cmd_reportBrokenSite");

    const allowedByPolicy = Services.policies.isAllowed(

      "DisableFeedbackCommands"

    );

    // Hide the items in base-browser. tor-browser#43903.

    const allowedByPolicy = false;

    cmd.toggleAttribute("hidden", !allowedByPolicy);

    const app = document.ownerGlobal.PanelMultiView.getViewNode(

      document,

toolkit/content/aboutSupport.js

+3 −2
Original line number Diff line number Diff line
@@ -1991,13 +1991,14 @@ function populateActionBox() { 
  if (ResetProfile.resetSupported()) {

    $("reset-box").style.display = "block";

  }

  if (!Services.appinfo.inSafeMode && AppConstants.platform !== "android") {

  // tor-browser#41755: Do not show safe mode, as it would disable NoScript.

  /*if (!Services.appinfo.inSafeMode && AppConstants.platform !== "android") {

    $("safe-mode-box").style.display = "block";



    if (Services.policies && !Services.policies.isAllowed("safeMode")) {

      $("restart-in-safe-mode-button").setAttribute("disabled", "true");

    }

  }

  }*/

}



// Prompt user to restart the browser in safe mode