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

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

Issue 1105263004: "Load image" context menu item to reload a LoFi image. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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
Index: chrome/browser/renderer_context_menu/render_view_context_menu.cc
diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
index 6946015097b93dacbf13a47ebc89046237c42918..d49e748f25bd40c3388d2517d683ba3cc32c02d4 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -216,8 +216,9 @@ const struct UmaEnumCommandIdPair {
{63, IDC_WRITING_DIRECTION_DEFAULT},
{64, IDC_WRITING_DIRECTION_LTR},
{65, IDC_WRITING_DIRECTION_RTL},
+ {66, IDC_CONTENT_CONTEXT_RELOAD_ORIGINAL_IMAGE},
Lei Zhang 2015/05/11 20:16:53 You need to update tools/metrics/histograms/histog
megjablon 2015/05/11 22:58:40 Done.
// Add new items here and use |enum_id| from the next line.
- {66, 0}, // Must be the last. Increment |enum_id| when new IDC was added.
+ {67, 0}, // Must be the last. Increment |enum_id| when new IDC was added.
};
// Collapses large ranges of ids before looking for UMA enum.
@@ -698,6 +699,11 @@ void RenderViewContextMenu::AppendImageItems() {
IDS_CONTENT_CONTEXT_COPYIMAGELOCATION);
menu_model_.AddItemWithStringId(IDC_CONTENT_CONTEXT_COPYIMAGE,
IDS_CONTENT_CONTEXT_COPYIMAGE);
+ if (params_.has_image_contents && params_.image_was_fetched_lo_fi) {
+ menu_model_.AddItemWithStringId(
+ IDC_CONTENT_CONTEXT_RELOAD_ORIGINAL_IMAGE,
+ IDS_CONTENT_CONTEXT_RELOAD_ORIGINAL_IMAGE);
+ }
DataReductionProxyChromeSettings* settings =
DataReductionProxyChromeSettingsFactory::GetForBrowserContext(
browser_context_);
@@ -1118,6 +1124,7 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const {
// The images shown in the most visited thumbnails can't be opened or
// searched for conventionally.
case IDC_CONTENT_CONTEXT_OPEN_ORIGINAL_IMAGE_NEW_TAB:
+ case IDC_CONTENT_CONTEXT_RELOAD_ORIGINAL_IMAGE:
case IDC_CONTENT_CONTEXT_OPENIMAGENEWTAB:
case IDC_CONTENT_CONTEXT_SEARCHWEBFORIMAGE:
return params_.src_url.is_valid() &&
@@ -1433,6 +1440,10 @@ void RenderViewContextMenu::ExecuteCommand(int id, int event_flags) {
data_reduction_proxy::kDataReductionPassThroughHeader);
break;
+ case IDC_CONTENT_CONTEXT_RELOAD_ORIGINAL_IMAGE:
+ ReloadOriginalImage();
+ break;
+
case IDC_CONTENT_CONTEXT_OPENIMAGENEWTAB:
case IDC_CONTENT_CONTEXT_OPENAVNEWTAB:
OpenURL(params_.src_url,
@@ -1787,6 +1798,14 @@ void RenderViewContextMenu::CopyImageAt(int x, int y) {
source_web_contents_->GetRenderViewHost()->CopyImageAt(x, y);
}
+void RenderViewContextMenu::ReloadOriginalImage() {
+ RenderFrameHost* render_frame_host = GetRenderFrameHost();
+ if (!render_frame_host)
+ return;
+ render_frame_host->Send(new ChromeViewMsg_RequestReloadImageForContextNode(
+ render_frame_host->GetRoutingID()));
+}
+
void RenderViewContextMenu::GetImageThumbnailForSearch() {
RenderFrameHost* render_frame_host = GetRenderFrameHost();
if (!render_frame_host)

Powered by Google App Engine
This is Rietveld 408576698