Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(56)

Unified Diff: chrome/browser/renderer_context_menu/render_view_context_menu.h

Issue 2035033003: Split RenderViewContextMenu code into functions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/renderer_context_menu/render_view_context_menu.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_context_menu/render_view_context_menu.h
diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.h b/chrome/browser/renderer_context_menu/render_view_context_menu.h
index 1ed0e7b656a31d88ab486d5fe15297a31fe86fff..62a0f7443d616fb8a2b3689d71a593897de2b348 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.h
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.h
@@ -148,19 +148,44 @@ class RenderViewContextMenu : public RenderViewContextMenuBase {
void AppendProtocolHandlerSubMenu();
void AppendPasswordItems();
- // Copy to the clipboard an image located at a point in the RenderView
- void CopyImageAt(int x, int y);
-
- // Load the original image located at a point in the RenderView.
- void LoadOriginalImage();
-
- // Get an image located at a point in the RenderView for search.
- void GetImageThumbnailForSearch();
-
- // Launch the inspector targeting a point in the RenderView
- void Inspect(int x, int y);
-
- void SaveLinkAs();
+ // Command enabled query functions.
+ bool IsReloadEnabled() const;
+ bool IsViewSourceEnabled() const;
+ bool IsDevCommandEnabled(int id) const;
+ bool IsTranslateEnabled() const;
+ bool IsSaveLinkAsEnabled() const;
+ bool IsSaveImageAsEnabled() const;
+ bool IsSaveAsEnabled() const;
+ bool IsSavePageEnabled() const;
+ bool IsPasteEnabled() const;
+ bool IsPasteAndMatchStyleEnabled() const;
+ bool IsRouteMediaEnabled() const;
+
+ // Command execution functions.
+ void ExecOpenLinkNewTab();
+ void ExecProtocolHandler(int event_flags, int handler_index);
+ void ExecOpenLinkInProfile(int profile_index);
+ void ExecInspectElement();
+ void ExecInspectBackgroundPage();
+ void ExecSaveLinkAs();
+ void ExecSaveAs();
+ void ExecCopyLinkText();
+ void ExecCopyImageAt();
+ void ExecSearchWebForImage();
+ void ExecLoadOriginalImage();
+ void ExecPlayPause();
+ void ExecMute();
+ void ExecLoop();
+ void ExecControls();
+ void ExecRotateCW();
+ void ExecRotateCCW();
+ void ExecReloadPackagedApp();
+ void ExecRestartPackagedApp();
+ void ExecPrint();
+ void ExecRouteMedia();
+ void ExecTranslate();
+ void ExecLanguageSettings(int event_flags);
+ void ExecProtocolHandlerSettings(int event_flags);
// Writes the specified text/url to the system clipboard
void WriteURLToClipboard(const GURL& url);
@@ -170,8 +195,6 @@ class RenderViewContextMenu : public RenderViewContextMenuBase {
void PluginActionAt(const gfx::Point& location,
const blink::WebPluginAction& action);
- bool IsDevCommandEnabled(int id) const;
-
// Returns a list of registered ProtocolHandlers that can handle the clicked
// on URL.
ProtocolHandlerRegistry::ProtocolHandlerList GetHandlersForLinkUrl();
« no previous file with comments | « no previous file | chrome/browser/renderer_context_menu/render_view_context_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698