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

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

Issue 8135017: Refactor downloads into a ProfileKeyedService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Incorporated comments. Created 9 years, 2 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/download/download_manager.h
diff --git a/content/browser/download/download_manager.h b/content/browser/download/download_manager.h
index 822c1f7d24911aef5791bafada80308dbe078e16..d16ad2a5a543cde3fcfb6d35b2e7e39334e93ce7 100644
--- a/content/browser/download/download_manager.h
+++ b/content/browser/download/download_manager.h
@@ -294,22 +294,24 @@ class CONTENT_EXPORT DownloadManager
DownloadManagerDelegate* delegate() const { return delegate_; }
+ // For testing only. May be called from tests indirectly (through
+ // other for testing only methods).
+ void SetDownloadManagerDelegate(DownloadManagerDelegate* delegate);
+
private:
typedef std::set<DownloadItem*> DownloadSet;
typedef base::hash_map<int64, DownloadItem*> DownloadMap;
// For testing.
friend class DownloadManagerTest;
- friend class MockDownloadManager;
friend class DownloadTest;
+ friend class MockDownloadManager;
friend class base::RefCountedThreadSafe<DownloadManager,
BrowserThread::DeleteOnUIThread>;
friend struct BrowserThread::DeleteOnThread<BrowserThread::UI>;
friend class DeleteTask<DownloadManager>;
- void set_delegate(DownloadManagerDelegate* delegate) { delegate_ = delegate; }
-
virtual ~DownloadManager();
// Called on the FILE thread to check the existence of a downloaded file.

Powered by Google App Engine
This is Rietveld 408576698