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 db76c141d404a3526c9ba57790d124f6a78b0b54..b937a0e560b39d60c36bdc89770c1d5ed473a411 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -87,6 +87,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) {}
 
@@ -177,13 +178,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;