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

Unified Diff: chrome/browser/history/download_database.h

Issue 10665049: Make DownloadHistory observe manager, items (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 5 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: chrome/browser/history/download_database.h
diff --git a/chrome/browser/history/download_database.h b/chrome/browser/history/download_database.h
index 502610b169fc2b0b0877a188a1f2dbba21f55425..673948b75723a57d503b2e55b10a54d35917b713 100644
--- a/chrome/browser/history/download_database.h
+++ b/chrome/browser/history/download_database.h
@@ -54,14 +54,8 @@ class DownloadDatabase {
// Create a new database entry for one download and return its primary db id.
int64 CreateDownload(const content::DownloadPersistentStoreInfo& info);
- // Remove a download from the database.
- void RemoveDownload(DownloadID db_handle);
-
- // Remove all completed downloads that started after |remove_begin|
- // (inclusive) and before |remove_end|. You may use null Time values
- // to do an unbounded delete in either direction. This function ignores
- // all downloads that are in progress or are waiting to be cancelled.
- bool RemoveDownloadsBetween(base::Time remove_begin, base::Time remove_end);
+ // Remove all |handles| from the database.
+ void RemoveDownloads(const std::set<int64>& handles);
protected:
// Returns the database for the functions in this interface.
@@ -78,9 +72,6 @@ class DownloadDatabase {
bool DropDownloadTable();
private:
- // TODO(rdsmith): Remove after http://crbug.com/96627 has been resolved.
- void CheckThread();
-
bool EnsureColumnExists(const std::string& name, const std::string& type);
bool owning_thread_set_;

Powered by Google App Engine
This is Rietveld 408576698