Make DownloadHistory observe manager, items

content/public/browser/download_manager.h

Index: content/public/browser/download_manager.h
diff --git a/content/public/browser/download_manager.h b/content/public/browser/download_manager.h
index 7c503141fad305f7a8b4f7da5c22f8a540b49aab..74a871e50057c59b45f86bdf4e37b10f53392f3a 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -83,6 +83,7 @@ class CONTENT_EXPORT DownloadManager
     // OnDownloadCreated().  TODO(<whoever>): When we've fully specified the
     // possible states of the DownloadItem in download_item.h and removed
     // ModelChanged, we should remove the caveat above.
+    // DO NOT create another download synchronously in OnDownloadCreated().
     virtual void OnDownloadCreated(
         DownloadManager* manager, DownloadItem* item) {}
 
@@ -145,13 +146,16 @@ class CONTENT_EXPORT DownloadManager
 
   // Called by the embedder, after creating the download manager, to let it know
   // about downloads from previous runs of the browser.
-  virtual void OnPersistentStoreQueryComplete(
-      std::vector<DownloadPersistentStoreInfo>* entries) = 0;
-
-  // Called by the embedder, in response to
-  // DownloadManagerDelegate::AddItemToPersistentStore.
-  virtual void OnItemAddedToPersistentStore(int32 download_id,
-                                            int64 db_handle) = 0;
+  virtual 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,
+      DownloadItem::DownloadState state,
+      bool opened) = 0;
 
   // The number of in progress (including paused) downloads.
   virtual int InProgressCount() const = 0;
@@ -167,9 +171,6 @@ class CONTENT_EXPORT DownloadManager
   // it is or state it's in.
   virtual DownloadItem* GetDownload(int id) = 0;
 
-  // Called when Save Page download is done.
-  virtual void SavePageDownloadFinished(DownloadItem* download) = 0;
-
  protected:
   virtual ~DownloadManager() {}