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

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

Issue 260043002: Cleanup: It is better to reuse the content API which already exist. (Closed) Base URL: http://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 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 | no next file » | 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.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 735c456254bad1afd0f4c022af4132dd2487b404..0fb672f10eccc55dca97fa6e50b02d63dafd3065 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -1541,24 +1541,8 @@ void RenderViewContextMenu::ExecuteCommand(int id, int event_flags) {
const GURL& referrer =
params_.frame_url.is_empty() ? params_.page_url : params_.frame_url;
const GURL& url = params_.src_url;
- int64 post_id = -1;
- if (url == source_web_contents_->GetURL()) {
- const NavigationEntry* entry =
- source_web_contents_->GetController().GetActiveEntry();
- if (entry)
- post_id = entry->GetPostID();
- }
- DownloadManager* dlm = BrowserContext::GetDownloadManager(profile_);
- scoped_ptr<DownloadUrlParameters> dl_params(
- DownloadUrlParameters::FromWebContents(source_web_contents_, url));
- dl_params->set_referrer(
- content::Referrer(referrer, params_.referrer_policy));
- dl_params->set_post_id(post_id);
- dl_params->set_prefer_cache(true);
- if (post_id >= 0)
- dl_params->set_method("POST");
- dl_params->set_prompt(true);
- dlm->DownloadUrl(dl_params.Pass());
+ source_web_contents_->SaveFrame(url, content::Referrer(
+ referrer, params_.referrer_policy));
break;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698