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

Unified Diff: content/browser/renderer_host/resource_dispatcher_host.h

Issue 7624031: Treat files downloaded from the address bar as "always safe" (including extensions per discussion... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 4 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: content/browser/renderer_host/resource_dispatcher_host.h
===================================================================
--- content/browser/renderer_host/resource_dispatcher_host.h (revision 98485)
+++ content/browser/renderer_host/resource_dispatcher_host.h (working copy)
@@ -77,8 +77,8 @@
ResourceMessageFilter* filter,
bool* message_was_ok);
- // Initiates a download from the browser process (as opposed to a resource
- // request from the renderer or another child process).
+ // Initiates a download by explicit request of the renderer, e.g. due to
+ // alt-clicking a link.
void BeginDownload(const GURL& url,
const GURL& referrer,
const DownloadSaveInfo& save_info,
@@ -401,10 +401,9 @@
const GURL& new_first_party_for_cookies);
void OnReleaseDownloadedFile(int request_id);
- // Creates ResourceDispatcherHostRequestInfo for a browser-initiated request
- // (a download or a page save). |download| should be true if the request
- // is a file download.
- ResourceDispatcherHostRequestInfo* CreateRequestInfoForBrowserRequest(
+ // Creates ResourceDispatcherHostRequestInfo for a download or page save.
+ // |download| should be true if the request is a file download.
+ ResourceDispatcherHostRequestInfo* CreateRequestInfo(
ResourceHandler* handler,
int child_id,
int route_id,
« no previous file with comments | « content/browser/download/download_state_info.cc ('k') | content/browser/renderer_host/resource_dispatcher_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698