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 b9cd2a9eecedfce55048cba489800e61eb14f447..afd264059d73258b099584eef1bee785e47d053b 100644 |
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
@@ -1086,11 +1086,9 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const { |
if (!local_state->GetBoolean(prefs::kAllowFileSelectionDialogs)) |
return false; |
- if (params_.media_type == WebContextMenuData::MediaTypeCanvas) |
- return true; |
- |
- return params_.src_url.is_valid() && |
- ProfileIOData::IsHandledProtocol(params_.src_url.scheme()); |
+ return params_.src_url.is_empty() || |
Avi (use Gerrit)
2014/09/02 22:33:12
Can you comment that "is_empty" means canvas? (It
zino
2014/09/03 01:45:58
I added comments.
Thank you.
|
+ (params_.src_url.is_valid() && |
+ ProfileIOData::IsHandledProtocol(params_.src_url.scheme())); |
} |
// The images shown in the most visited thumbnails can't be opened or |
@@ -1345,11 +1343,12 @@ void RenderViewContextMenu::ExecuteCommand(int id, int event_flags) { |
case IDC_CONTENT_CONTEXT_SAVEAVAS: |
case IDC_CONTENT_CONTEXT_SAVEIMAGEAS: { |
- if (params_.media_type == WebContextMenuData::MediaTypeCanvas) { |
+ if (params_.media_type == WebContextMenuData::MediaTypeCanvas || |
+ (params_.media_type == WebContextMenuData::MediaTypeImage && |
+ params_.src_url.is_empty())) { |
source_web_contents_->GetRenderViewHost()->SaveImageAt( |
params_.x, params_.y); |
} else { |
- // TODO(zino): We can use SaveImageAt() like a case of canvas. |
RecordDownloadSource(DOWNLOAD_INITIATED_BY_CONTEXT_MENU); |
const GURL& url = params_.src_url; |
content::Referrer referrer = CreateSaveAsReferrer(url, params_); |