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

Unified Diff: content/public/browser/download_manager.h

Issue 23496076: WIP - Refactor programmatic downloads Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/public/browser/download_manager.h
diff --git a/content/public/browser/download_manager.h b/content/public/browser/download_manager.h
index 37c6d09da908f41dfdd78294aa55213577f244e8..3700835bffa232bc1ce6a94d04fec3ec8f0282b3 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -101,14 +101,12 @@ class CONTENT_EXPORT DownloadManager : public base::SupportsUserData::Data {
// clearing |downloads| first.
virtual void GetAllDownloads(DownloadVector* downloads) = 0;
- // Called by a download source (Currently DownloadResourceHandler)
- // to initiate the non-source portions of a download.
- // Returns the id assigned to the download. If the DownloadCreateInfo
- // specifies an id, that id will be used.
- virtual void StartDownload(
- scoped_ptr<DownloadCreateInfo> info,
- scoped_ptr<ByteStreamReader> stream,
- const DownloadUrlParameters::OnStartedCallback& on_started) = 0;
+ // Handles a download that's already in progress. The |request_handle|
+ // provides a handle for controlling the download. |info| specifies parameters
+ // for the new download including the ByteStreamReader.
+ virtual void StartDownloadWithActiveRequest(
+ scoped_ptr<DownloadRequestHandle> request_handle,
+ scoped_ptr<DownloadCreateInfo> info) = 0;
// Remove downloads after remove_begin (inclusive) and before remove_end
// (exclusive). You may pass in null Time values to do an unbounded delete
« no previous file with comments | « content/public/browser/download_interrupt_reasons.h ('k') | content/public/browser/download_url_parameters.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698