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

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

Issue 10915180: Make DownloadHistory observe manager, items (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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/browser/download/download_manager_impl.h
diff --git a/content/browser/download/download_manager_impl.h b/content/browser/download/download_manager_impl.h
index 7b8971a38d18a835725e94387d2af73a4acc68fd..fe25b2412d3e281dd79237fc1b2e6104851169c0 100644
--- a/content/browser/download/download_manager_impl.h
+++ b/content/browser/download/download_manager_impl.h
@@ -76,10 +76,16 @@ class CONTENT_EXPORT DownloadManagerImpl
scoped_ptr<content::DownloadUrlParameters> params) OVERRIDE;
virtual void AddObserver(Observer* observer) OVERRIDE;
virtual void RemoveObserver(Observer* observer) OVERRIDE;
- virtual void OnPersistentStoreQueryComplete(
- std::vector<content::DownloadPersistentStoreInfo>* entries) OVERRIDE;
- virtual void OnItemAddedToPersistentStore(int32 download_id,
- int64 db_handle) OVERRIDE;
+ virtual content::DownloadItem* CreateDownloadItem(
+ const FilePath& path,
+ const GURL& url,
+ const GURL& referrer_url,
+ const base::Time& start_time,
+ const base::Time& end_time,
+ int64 received_bytes,
+ int64 total_bytes,
+ content::DownloadItem::DownloadState state,
+ bool opened) OVERRIDE;
virtual int InProgressCount() const OVERRIDE;
virtual content::BrowserContext* GetBrowserContext() const OVERRIDE;
virtual void CheckForHistoryFilesRemoval() OVERRIDE;
@@ -140,9 +146,6 @@ class CONTENT_EXPORT DownloadManagerImpl
// containers; no-op if NDEBUG.
void AssertContainersConsistent() const;
- // Add a DownloadItem to history_downloads_.
- void AddDownloadItemToHistory(DownloadItemImpl* item, int64 db_handle);
-
// Remove from internal maps.
int RemoveDownloadItems(const DownloadItemImplVector& pending_deletes);
@@ -162,12 +165,6 @@ class CONTENT_EXPORT DownloadManagerImpl
content::DownloadDangerType danger_type,
const FilePath& intermediate_path);
- // Called when a download entry is committed to the persistent store.
- void OnDownloadItemAddedToPersistentStore(DownloadItemImpl* item);
-
- // Called when Save Page As entry is committed to the persistent store.
- void OnSavePageItemAddedToPersistentStore(DownloadItemImpl* item);
-
// Overridden from DownloadItemImplDelegate
// (Note that |GetBrowserContext| are present in both interfaces.)
virtual DownloadFileManager* GetDownloadFileManager() OVERRIDE;
@@ -181,7 +178,6 @@ class CONTENT_EXPORT DownloadManagerImpl
virtual void DownloadRemoved(DownloadItemImpl* download) OVERRIDE;
virtual void DownloadRenamedToIntermediateName(
DownloadItemImpl* download) OVERRIDE;
- virtual void DownloadRenamedToFinalName(DownloadItemImpl* download) OVERRIDE;
virtual void AssertStateConsistent(DownloadItemImpl* download) const OVERRIDE;
// Factory for creation of downloads items.

Powered by Google App Engine
This is Rietveld 408576698