Download code cleanup:

- choose better names for some helper methods
- move code to less random places

This change also adds bigger tuples support, up to Tuple8 in base/tuple.h.
The plan is to stop using such big number of parameters, but for now it's not
trivial.

This change also fixes some UI tests, not sure why it is so.

TEST=unit_tests, browser_tests, ui_tests
BUG=48913

Review URL: http://codereview.chromium.org/2805091

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53053 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 22 insertions, 0 deletions

diff --git a/chrome/browser/download/download_util.h b/chrome/browser/download/download_util.h
index 6ee8f89..72af5d5 100644
--- a/chrome/browser/download/download_util.h
+++ b/chrome/browser/download/download_util.h
@@ -25,7 +25,12 @@ class BaseDownloadItemModel;
 class DictionaryValue;
 class DownloadItem;
 class FilePath;
+class GURL;
+class ResourceDispatcherHost;
 class SkBitmap;
+class URLRequestContextGetter;
+
+struct DownloadSaveInfo;
 
 namespace download_util {
 
@@ -157,6 +162,23 @@ void AppendNumberToPath(FilePath* path, int number);
 // a number, -1 is returned.
 int GetUniquePathNumber(const FilePath& path);
 
+// Download the URL. Must be called on the IO thread.
+void DownloadUrl(const GURL& url,
+                 const GURL& referrer,
+                 const std::string& referrer_charset,
+                 const DownloadSaveInfo& save_info,
+                 ResourceDispatcherHost* rdh,
+                 int render_process_host_id,
+                 int render_view_id,
+                 URLRequestContextGetter* request_context_getter);
+
+// Tells the resource dispatcher host to cancel a download request.
+// Must be called on the IO thread.
+void CancelDownloadRequest(ResourceDispatcherHost* rdh,
+                           int render_process_id,
+                           int request_id);
+
+
 }  // namespace download_util
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_UTIL_H_