Remove duplicated separator in page context menu

fixes brave/brave-browser#15714

RemoveAdjacentSeparators::RenderViewContextMenuBase is used

browser/ui/BUILD.gn

@@ -151,6 +151,8 @@ source_set("ui") {
       "views/speedreader/speedreader_bubble_util.h",
       "views/speedreader/speedreader_mode_bubble.cc",
       "views/speedreader/speedreader_mode_bubble.h",
+      "views/tab_contents/brave_web_contents_view_delegate_views.cc",
+      "views/tab_contents/brave_web_contents_view_delegate_views.h",
       "views/tabs/brave_browser_tab_strip_controller.cc",
       "views/tabs/brave_browser_tab_strip_controller.h",
       "views/tabs/brave_tab_context_menu_contents.cc",

browser/ui/views/tab_contents/brave_web_contents_view_delegate_views.cc

@@ -0,0 +1,25 @@
+/* Copyright (c) 2021 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#include "brave/browser/ui/views/tab_contents/brave_web_contents_view_delegate_views.h"
+
+#include <memory>
+#include <utility>
+
+#include "chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.h"
+#include "components/renderer_context_menu/render_view_context_menu_base.h"
+
+BraveWebContentsViewDelegateViews::BraveWebContentsViewDelegateViews(
+    content::WebContents* web_contents)
+    : ChromeWebContentsViewDelegateViews(web_contents) {}
+
+BraveWebContentsViewDelegateViews::~BraveWebContentsViewDelegateViews() =
+    default;
+
+void BraveWebContentsViewDelegateViews::ShowMenu(
+    std::unique_ptr<RenderViewContextMenuBase> menu) {
+  menu->RemoveAdjacentSeparators();
+  ChromeWebContentsViewDelegateViews::ShowMenu(std::move(menu));
+}

browser/ui/views/tab_contents/brave_web_contents_view_delegate_views.h

@@ -0,0 +1,29 @@
+/* Copyright (c) 2021 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#ifndef BRAVE_BROWSER_UI_VIEWS_TAB_CONTENTS_BRAVE_WEB_CONTENTS_VIEW_DELEGATE_VIEWS_H_
+#define BRAVE_BROWSER_UI_VIEWS_TAB_CONTENTS_BRAVE_WEB_CONTENTS_VIEW_DELEGATE_VIEWS_H_
+
+#include <memory>
+
+#include "chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.h"
+
+class RenderViewContextMenuBase;
+
+namespace content {
+class WebContents;
+}
+
+class BraveWebContentsViewDelegateViews
+    : public ChromeWebContentsViewDelegateViews {
+ public:
+  explicit BraveWebContentsViewDelegateViews(
+      content::WebContents* web_contents);
+  ~BraveWebContentsViewDelegateViews() override;
+
+  void ShowMenu(std::unique_ptr<RenderViewContextMenuBase> menu) override;
+};
+
+#endif  // BRAVE_BROWSER_UI_VIEWS_TAB_CONTENTS_BRAVE_WEB_CONTENTS_VIEW_DELEGATE_VIEWS_H_

chromium_src/chrome/browser/DEPS

@@ -64,6 +64,7 @@ include_rules = [
   "+../../../../../../../chrome/browser/ui/views/location_bar",
   "+../../../../../../../chrome/browser/ui/views/page_action",
   "+../../../../../../../chrome/browser/ui/views/profiles",
+  "+../../../../../../../chrome/browser/ui/views/tab_contents",
   "+../../../../../../../chrome/browser/ui/views/tabs",
   "+../../../../../../../chrome/browser/ui/views/toolbar",
   "+../../../../../../../chrome/browser/ui/views/translate",

chromium_src/chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc

@@ -0,0 +1,15 @@
+/* Copyright (c) 2021 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#include "brave/browser/ui/views/tab_contents/brave_web_contents_view_delegate_views.h"
+
+#define CreateWebContentsViewDelegate CreateWebContentsViewDelegate_ChromiumImpl
+#include "../../../../../../../chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc"
+#undef CreateWebContentsViewDelegate
+
+content::WebContentsViewDelegate* CreateWebContentsViewDelegate(
+    content::WebContents* web_contents) {
+  return new BraveWebContentsViewDelegateViews(web_contents);
+}