From 1d53a448947384b6b4f7f4f0d7b415a3655d25f5 Mon Sep 17 00:00:00 2001 From: Max Karolinskiy Date: Thu, 18 Oct 2018 00:23:25 -0400 Subject: [PATCH] Hide RenderViewContextMenu::AddSpellCheckServiceItem method with one in BraveRenderViewContextMenu that does not add Ask Brave menu item. Fixes brave/brave-browser#1623 --- .../renderer_context_menu/render_view_context_menu.cc | 7 +++++++ .../renderer_context_menu/render_view_context_menu.h | 2 ++ 2 files changed, 9 insertions(+) diff --git a/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.cc index 3f966274d847..5bc0bfdc8961 100644 --- a/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.cc +++ b/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.cc @@ -62,3 +62,10 @@ void BraveRenderViewContextMenu::ExecuteCommand(int id, int event_flags) { RenderViewContextMenu_Chromium::ExecuteCommand(id, event_flags); } } + +// static +void BraveRenderViewContextMenu::AddSpellCheckServiceItem( + ui::SimpleMenuModel* menu, + bool is_checked) { + // Suppress adding "Spellcheck->Ask Brave for suggestions" item +} diff --git a/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.h b/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.h index ffe438e4af3e..6568cc39d6e8 100644 --- a/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.h +++ b/chromium_src/chrome/browser/renderer_context_menu/render_view_context_menu.h @@ -18,6 +18,8 @@ class BraveRenderViewContextMenu : public RenderViewContextMenu_Chromium { void AppendBraveLinkItems() override; bool IsCommandIdEnabled(int command_id) const override; void ExecuteCommand(int id, int event_flags) override; + static void AddSpellCheckServiceItem(ui::SimpleMenuModel* menu, + bool is_checked); }; // Use our own subclass as the real RenderViewContextMenu.