Skip to content
Snippets Groups Projects
Commit bf077d52 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 bf1d3690
Branches
No related tags found
1 merge request!1500TB 43415: Rebased onto 134.0a1
......@@ -419,6 +419,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"
#ifdef XP_MACOSX
......@@ -432,12 +433,15 @@
hidden="true"
appmenu-data-l10n-id="menu-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"
hidden="true"
data-l10n-id="menu-help-more-troubleshooting-info"
appmenu-data-l10n-id="appmenu-help-more-troubleshooting-info"/>
<menuitem id="help_reportSiteIssue"
......@@ -455,6 +459,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"/>
......
......@@ -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
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment