Hide RenderViewContextMenu::AddSpellCheckServiceItem method with

one in BraveRenderViewContextMenu that does not add Ask Brave menu item. Fixes brave/brave-browser#1623
brave · Oct 18, 2018 · 1d53a44 · 1d53a44
1 parent d3bc8e0
commit 1d53a44
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
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
@@ -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
@@ -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.