2021-04-09 01:15:51 +02:00
|
|
|
diff --git chrome/browser/renderer_context_menu/render_view_context_menu.cc chrome/browser/renderer_context_menu/render_view_context_menu.cc
|
2021-08-20 01:40:49 +02:00
|
|
|
index 9e358b1cde224..67ccd6ca0c50d 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- chrome/browser/renderer_context_menu/render_view_context_menu.cc
|
|
|
|
+++ chrome/browser/renderer_context_menu/render_view_context_menu.cc
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -279,6 +279,13 @@ base::OnceCallback<void(RenderViewContextMenu*)>* GetMenuShownCallback() {
|
2021-04-09 01:15:51 +02:00
|
|
|
return callback.get();
|
|
|
|
}
|
|
|
|
|
|
|
|
+
|
|
|
|
+RenderViewContextMenu::MenuCreatedCallback* GetMenuCreatedCallback() {
|
|
|
|
+ static base::NoDestructor<RenderViewContextMenu::MenuCreatedCallback>
|
|
|
|
+ callback;
|
|
|
|
+ return callback.get();
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
enum class UmaEnumIdLookupType {
|
|
|
|
GeneralEnumId,
|
|
|
|
ContextSpecificEnumId,
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -490,6 +497,10 @@ int FindUMAEnumValueForCommand(int id, UmaEnumIdLookupType type) {
|
2021-04-09 01:15:51 +02:00
|
|
|
if (ContextMenuMatcher::IsExtensionsCustomCommandId(id))
|
|
|
|
return 1;
|
|
|
|
|
|
|
|
+ // Match the MENU_ID_USER_FIRST to MENU_ID_USER_LAST range from cef_types.h.
|
|
|
|
+ if (id >= 26500 && id <= 28500)
|
|
|
|
+ return 1;
|
|
|
|
+
|
|
|
|
id = CollapseCommandsForUMA(id);
|
|
|
|
const auto& map = GetIdcToUmaMap(type);
|
|
|
|
auto it = map.find(id);
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -666,6 +677,14 @@ RenderViewContextMenu::RenderViewContextMenu(
|
2021-07-23 18:40:13 +02:00
|
|
|
system_app_type_ = GetBrowser() && GetBrowser()->app_controller()
|
|
|
|
? GetBrowser()->app_controller()->system_app_type()
|
|
|
|
: absl::nullopt;
|
2021-04-09 01:15:51 +02:00
|
|
|
+
|
|
|
|
+ auto* cb = GetMenuCreatedCallback();
|
|
|
|
+ if (!cb->is_null()) {
|
|
|
|
+ first_observer_ = cb->Run(this);
|
|
|
|
+ if (first_observer_) {
|
|
|
|
+ observers_.AddObserver(first_observer_.get());
|
|
|
|
+ }
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
|
|
|
RenderViewContextMenu::~RenderViewContextMenu() = default;
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -1032,6 +1051,12 @@ void RenderViewContextMenu::InitMenu() {
|
2021-04-09 01:15:51 +02:00
|
|
|
// menu, meaning that each menu item added/removed in this function will cause
|
|
|
|
// it to visibly jump on the screen (see b/173569669).
|
|
|
|
AppendQuickAnswersItems();
|
|
|
|
+
|
|
|
|
+ if (first_observer_) {
|
|
|
|
+ // Do this last so that the observer can optionally modify previously
|
|
|
|
+ // created items.
|
|
|
|
+ first_observer_->InitMenu(params_);
|
|
|
|
+ }
|
|
|
|
}
|
|
|
|
|
|
|
|
Profile* RenderViewContextMenu::GetProfile() const {
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -2816,6 +2841,12 @@ void RenderViewContextMenu::RegisterMenuShownCallbackForTesting(
|
2021-04-09 01:15:51 +02:00
|
|
|
*GetMenuShownCallback() = std::move(cb);
|
|
|
|
}
|
|
|
|
|
|
|
|
+// static
|
|
|
|
+void RenderViewContextMenu::RegisterMenuCreatedCallback(
|
|
|
|
+ MenuCreatedCallback cb) {
|
|
|
|
+ *GetMenuCreatedCallback() = cb;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
ProtocolHandlerRegistry::ProtocolHandlerList
|
|
|
|
RenderViewContextMenu::GetHandlersForLinkUrl() {
|
|
|
|
ProtocolHandlerRegistry::ProtocolHandlerList handlers =
|
|
|
|
diff --git chrome/browser/renderer_context_menu/render_view_context_menu.h chrome/browser/renderer_context_menu/render_view_context_menu.h
|
2021-08-20 01:40:49 +02:00
|
|
|
index fbcde566b8c90..261b1b3e05dfc 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- chrome/browser/renderer_context_menu/render_view_context_menu.h
|
|
|
|
+++ chrome/browser/renderer_context_menu/render_view_context_menu.h
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -106,6 +106,12 @@ class RenderViewContextMenu : public RenderViewContextMenuBase,
|
2021-04-09 01:15:51 +02:00
|
|
|
static void RegisterMenuShownCallbackForTesting(
|
|
|
|
base::OnceCallback<void(RenderViewContextMenu*)> cb);
|
|
|
|
|
|
|
|
+ // Registers a callback that will be called each time a context menu is
|
|
|
|
+ // created.
|
|
|
|
+ using MenuCreatedCallback = base::RepeatingCallback<
|
|
|
|
+ std::unique_ptr<RenderViewContextMenuObserver>(RenderViewContextMenu*)>;
|
|
|
|
+ static void RegisterMenuCreatedCallback(MenuCreatedCallback cb);
|
|
|
|
+
|
|
|
|
protected:
|
|
|
|
Profile* GetProfile() const;
|
|
|
|
|
2021-08-20 01:40:49 +02:00
|
|
|
@@ -304,6 +310,9 @@ class RenderViewContextMenu : public RenderViewContextMenuBase,
|
2021-07-23 18:40:13 +02:00
|
|
|
// built.
|
|
|
|
bool is_protocol_submenu_valid_ = false;
|
2021-04-09 01:15:51 +02:00
|
|
|
|
|
|
|
+ // An observer returned via MenuCreatedCallback that will be called first.
|
|
|
|
+ std::unique_ptr<RenderViewContextMenuObserver> first_observer_;
|
|
|
|
+
|
|
|
|
// An observer that handles spelling suggestions, "Add to dictionary", and
|
|
|
|
// "Use enhanced spell check" items.
|
|
|
|
std::unique_ptr<SpellingMenuObserver> spelling_suggestions_menu_observer_;
|
|
|
|
diff --git chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
|
2021-07-23 18:40:13 +02:00
|
|
|
index 13a65de79e185..0901c4cade4f1 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
|
|
|
|
+++ chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
|
|
|
|
@@ -136,6 +136,9 @@ void RenderViewContextMenuViews::RunMenuAt(views::Widget* parent,
|
|
|
|
bool RenderViewContextMenuViews::GetAcceleratorForCommandId(
|
|
|
|
int command_id,
|
|
|
|
ui::Accelerator* accel) const {
|
|
|
|
+ if (RenderViewContextMenu::GetAcceleratorForCommandId(command_id, accel))
|
|
|
|
+ return true;
|
|
|
|
+
|
|
|
|
// There are no formally defined accelerators we can query so we assume
|
|
|
|
// that Ctrl+C, Ctrl+V, Ctrl+X, Ctrl-A, etc do what they normally do.
|
|
|
|
switch (command_id) {
|
|
|
|
diff --git components/renderer_context_menu/render_view_context_menu_base.cc components/renderer_context_menu/render_view_context_menu_base.cc
|
2021-08-20 01:40:49 +02:00
|
|
|
index f40f15c23bfb2..06d4d197ce137 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- components/renderer_context_menu/render_view_context_menu_base.cc
|
|
|
|
+++ components/renderer_context_menu/render_view_context_menu_base.cc
|
2021-06-04 03:34:56 +02:00
|
|
|
@@ -376,6 +376,17 @@ bool RenderViewContextMenuBase::IsCommandIdChecked(int id) const {
|
2021-04-09 01:15:51 +02:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
+bool RenderViewContextMenuBase::GetAcceleratorForCommandId(
|
|
|
|
+ int id,
|
|
|
|
+ ui::Accelerator* accelerator) const {
|
|
|
|
+ for (auto& observer : observers_) {
|
|
|
|
+ if (observer.IsCommandIdSupported(id))
|
|
|
|
+ return observer.GetAccelerator(id, accelerator);
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ return false;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
void RenderViewContextMenuBase::ExecuteCommand(int id, int event_flags) {
|
|
|
|
command_executed_ = true;
|
|
|
|
RecordUsedItem(id);
|
|
|
|
diff --git components/renderer_context_menu/render_view_context_menu_base.h components/renderer_context_menu/render_view_context_menu_base.h
|
2021-06-04 03:34:56 +02:00
|
|
|
index 051953091fd4f..7e771526ce52a 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- components/renderer_context_menu/render_view_context_menu_base.h
|
|
|
|
+++ components/renderer_context_menu/render_view_context_menu_base.h
|
2021-06-04 03:34:56 +02:00
|
|
|
@@ -82,6 +82,9 @@ class RenderViewContextMenuBase : public ui::SimpleMenuModel::Delegate,
|
2021-04-09 01:15:51 +02:00
|
|
|
|
|
|
|
const ui::SimpleMenuModel& menu_model() const { return menu_model_; }
|
|
|
|
const content::ContextMenuParams& params() const { return params_; }
|
|
|
|
+ content::WebContents* source_web_contents() const {
|
|
|
|
+ return source_web_contents_;
|
|
|
|
+ }
|
|
|
|
|
|
|
|
// Returns true if the specified command id is known and valid for
|
|
|
|
// this menu. If the command is known |enabled| is set to indicate
|
2021-06-04 03:34:56 +02:00
|
|
|
@@ -90,6 +93,9 @@ class RenderViewContextMenuBase : public ui::SimpleMenuModel::Delegate,
|
2021-04-09 01:15:51 +02:00
|
|
|
|
|
|
|
// SimpleMenuModel::Delegate implementation.
|
|
|
|
bool IsCommandIdChecked(int command_id) const override;
|
|
|
|
+ bool GetAcceleratorForCommandId(
|
|
|
|
+ int command_id,
|
|
|
|
+ ui::Accelerator* accelerator) const override;
|
|
|
|
void ExecuteCommand(int command_id, int event_flags) override;
|
|
|
|
void OnMenuWillShow(ui::SimpleMenuModel* source) override;
|
|
|
|
void MenuClosed(ui::SimpleMenuModel* source) override;
|
2021-06-04 03:34:56 +02:00
|
|
|
@@ -120,6 +126,9 @@ class RenderViewContextMenuBase : public ui::SimpleMenuModel::Delegate,
|
2021-04-09 01:15:51 +02:00
|
|
|
content::WebContents* GetWebContents() const override;
|
|
|
|
content::BrowserContext* GetBrowserContext() const override;
|
|
|
|
|
|
|
|
+ // May return nullptr if the frame was deleted while the menu was open.
|
|
|
|
+ content::RenderFrameHost* GetRenderFrameHost() const;
|
|
|
|
+
|
|
|
|
protected:
|
|
|
|
friend class RenderViewContextMenuTest;
|
|
|
|
friend class RenderViewContextMenuPrefsTest;
|
2021-06-04 03:34:56 +02:00
|
|
|
@@ -157,9 +166,6 @@ class RenderViewContextMenuBase : public ui::SimpleMenuModel::Delegate,
|
2021-04-09 01:15:51 +02:00
|
|
|
// TODO(oshima): Remove this.
|
|
|
|
virtual void AppendPlatformEditableItems() {}
|
|
|
|
|
|
|
|
- // May return nullptr if the frame was deleted while the menu was open.
|
|
|
|
- content::RenderFrameHost* GetRenderFrameHost() const;
|
|
|
|
-
|
|
|
|
bool IsCustomItemChecked(int id) const;
|
|
|
|
bool IsCustomItemEnabled(int id) const;
|
|
|
|
|
|
|
|
diff --git components/renderer_context_menu/render_view_context_menu_observer.cc components/renderer_context_menu/render_view_context_menu_observer.cc
|
2021-04-21 00:52:34 +02:00
|
|
|
index 2e2d05f91c646..85b256b2be9bd 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- components/renderer_context_menu/render_view_context_menu_observer.cc
|
|
|
|
+++ components/renderer_context_menu/render_view_context_menu_observer.cc
|
|
|
|
@@ -15,3 +15,8 @@ bool RenderViewContextMenuObserver::IsCommandIdChecked(int command_id) {
|
|
|
|
bool RenderViewContextMenuObserver::IsCommandIdEnabled(int command_id) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
+
|
|
|
|
+bool RenderViewContextMenuObserver::GetAccelerator(int command_id,
|
|
|
|
+ ui::Accelerator* accel) {
|
|
|
|
+ return false;
|
|
|
|
+}
|
|
|
|
diff --git components/renderer_context_menu/render_view_context_menu_observer.h components/renderer_context_menu/render_view_context_menu_observer.h
|
2021-04-21 00:52:34 +02:00
|
|
|
index b360a8eb4e820..6f9023a629046 100644
|
2021-04-09 01:15:51 +02:00
|
|
|
--- components/renderer_context_menu/render_view_context_menu_observer.h
|
|
|
|
+++ components/renderer_context_menu/render_view_context_menu_observer.h
|
|
|
|
@@ -11,6 +11,10 @@ namespace content {
|
|
|
|
struct ContextMenuParams;
|
|
|
|
}
|
|
|
|
|
|
|
|
+namespace ui {
|
|
|
|
+class Accelerator;
|
|
|
|
+}
|
|
|
|
+
|
|
|
|
// The interface used for implementing context-menu items. The following
|
|
|
|
// instruction describe how to implement a context-menu item with this
|
|
|
|
// interface.
|
|
|
|
@@ -100,6 +104,8 @@ class RenderViewContextMenuObserver {
|
|
|
|
virtual bool IsCommandIdChecked(int command_id);
|
|
|
|
virtual bool IsCommandIdEnabled(int command_id);
|
|
|
|
|
|
|
|
+ virtual bool GetAccelerator(int command_id, ui::Accelerator* accel);
|
|
|
|
+
|
|
|
|
// Called when a user selects the specified context-menu item. This is
|
|
|
|
// only called when the observer returns true for IsCommandIdSupported()
|
|
|
|
// for that |command_id|.
|