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

Unified Diff: chrome/browser/download/download_service.h

Issue 10915180: Make DownloadHistory observe manager, items (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: @r166491 Created 8 years, 1 month 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: chrome/browser/download/download_service.h
diff --git a/chrome/browser/download/download_service.h b/chrome/browser/download/download_service.h
index 0516a0ecf4e9a861f93dd59fd699c22d1780ba48..7f853e4489b7e9b87eaf32048d035c433322ced2 100644
--- a/chrome/browser/download/download_service.h
+++ b/chrome/browser/download/download_service.h
@@ -10,9 +10,11 @@
#include "base/basictypes.h"
#include "base/callback_forward.h"
#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
#include "chrome/browser/profiles/profile_keyed_service.h"
class ChromeDownloadManagerDelegate;
+class DownloadHistory;
class Profile;
namespace content {
@@ -33,6 +35,9 @@ class DownloadService : public ProfileKeyedService {
// Get the download manager delegate, creating it if it doesn't already exist.
ChromeDownloadManagerDelegate* GetDownloadManagerDelegate();
+ // Get the interface to the history system.
+ DownloadHistory* GetDownloadHistory();
Randy Smith (Not in Mondays) 2012/11/07 21:10:29 Specify may be null if OTR.
benjhayden 2012/11/08 18:57:04 Done.
+
// Has a download manager been created?
bool HasCreatedDownloadManager();
@@ -61,6 +66,8 @@ class DownloadService : public ProfileKeyedService {
// callbacks.
scoped_refptr<ChromeDownloadManagerDelegate> manager_delegate_;
+ scoped_ptr<DownloadHistory> download_history_;
+
std::vector<OnManagerCreatedCallback> on_manager_created_callbacks_;
DISALLOW_COPY_AND_ASSIGN(DownloadService);

Powered by Google App Engine
This is Rietveld 408576698