tor-browser

commit 49f28e9329baffe51ff9d24f8d47c89651a86d44
parent 2cc9a69038bc1283487441f3af64cbc6ccad605a
Author: Sarah Clements <sclements@mozilla.com>
Date:   Fri, 14 Nov 2025 15:43:25 +0000

Bug 1998360 - Ensure that Remove from Sidebar context menu only applies to extensions r=nsharpley

Differential Revision: https://phabricator.services.mozilla.com/D272164

Diffstat:
M
browser/components/sidebar/sidebar-main.mjs
 | 
2
+-
M
browser/components/sidebar/tests/browser/browser_sidebar_context_menu.js
 | 
4
++++

2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/browser/components/sidebar/sidebar-main.mjs b/browser/components/sidebar/sidebar-main.mjs
@@ -399,7 +399,7 @@ export default class SidebarMain extends MozLitElement {
     this._manageExtensionMenuItem.hidden = true;
     this._removeExtensionMenuItem.hidden = true;
     this._reportExtensionMenuItem.hidden = true;
-    this._unpinExtensionMenuItem.hidden = false;
+    this._unpinExtensionMenuItem.hidden = true;
     this._customizeSidebarMenuItem.hidden = false;
     this._enableVerticalTabsMenuItem.hidden = false;
     this._hideSidebarMenuItem.hidden = false;
diff --git a/browser/components/sidebar/tests/browser/browser_sidebar_context_menu.js b/browser/components/sidebar/tests/browser/browser_sidebar_context_menu.js
@@ -213,6 +213,10 @@ add_task(async function test_sidebar_context_menu() {
     document.getElementById("sidebar-context-menu-manage-extension").hidden,
     "Manage extension button is hidden"
   );
+  ok(
+    document.getElementById("sidebar-context-menu-unpin-extension").hidden,
+    "Remove extension from Sidebar button is hidden"
+  );
   contextMenu.hidePopup();
 
   await Promise.all([