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

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

Issue 10861002: Revert 152213 - Replace the DownloadFileManager with direct ownership of DownloadFile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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
« no previous file with comments | « content/public/browser/download_destination_observer.h ('k') | content/public/test/mock_download_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/download_manager.h
===================================================================
--- content/public/browser/download_manager.h (revision 152281)
+++ content/public/browser/download_manager.h (working copy)
@@ -132,10 +132,33 @@
scoped_ptr<DownloadCreateInfo> info,
scoped_ptr<ByteStreamReader> stream) = 0;
+ // Notifications sent from the download thread to the UI thread
+ virtual void UpdateDownload(int32 download_id,
+ int64 bytes_so_far,
+ int64 bytes_per_sec,
+ const std::string& hash_state) = 0;
+
+ // |download_id| is the ID of the download.
+ // |size| is the number of bytes that have been downloaded.
+ // |hash| is sha256 hash for the downloaded file. It is empty when the hash
+ // is not available.
+ virtual void OnResponseCompleted(int32 download_id, int64 size,
+ const std::string& hash) = 0;
+
// Offthread target for cancelling a particular download. Will be a no-op
// if the download has already been cancelled.
virtual void CancelDownload(int32 download_id) = 0;
+ // Called when there is an error in the download.
+ // |download_id| is the ID of the download.
+ // |size| is the number of bytes that are currently downloaded.
+ // |hash_state| is the current state of the hash of the data that has been
+ // downloaded.
+ // |reason| is a download interrupt reason code.
+ virtual void OnDownloadInterrupted(
+ int32 download_id,
+ DownloadInterruptReason reason) = 0;
+
// Remove downloads after remove_begin (inclusive) and before remove_end
// (exclusive). You may pass in null Time values to do an unbounded delete
// in either direction.
« no previous file with comments | « content/public/browser/download_destination_observer.h ('k') | content/public/test/mock_download_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698