Merge pull request #24428 from brave/maxk-hide-history-groups-menu-item

Hides Grouped History menu item.
brave · Jul 2, 2024 · 6180543 · 6180543
1 parent 2503bbe
commit 6180543
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc b/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc
@@ -53,7 +53,13 @@ const char kBraveSyncedTabsUrl[] = "brave://history/syncedTabs";
 BraveRecentTabsSubMenuModel::BraveRecentTabsSubMenuModel(
     ui::AcceleratorProvider* accelerator_provider,
     Browser* browser)
-    : RecentTabsSubMenuModel(accelerator_provider, browser) {}
+    : RecentTabsSubMenuModel(accelerator_provider, browser) {
+  // We disable history clusters feature, so this command won't work.
+  std::optional<size_t> show_history_clusters_index =
+      GetIndexOfCommandId(IDC_SHOW_HISTORY_CLUSTERS_SIDE_PANEL);
+  CHECK(show_history_clusters_index);
+  RemoveItemAt(show_history_clusters_index.value());
+}
 
 BraveRecentTabsSubMenuModel::~BraveRecentTabsSubMenuModel() {}