Support for adding save page download items into downloads history.

chrome/browser/download/download_manager.h

Index: chrome/browser/download/download_manager.h
===================================================================
--- chrome/browser/download/download_manager.h	(revision 91883)
+++ chrome/browser/download/download_manager.h	(working copy)
@@ -266,6 +266,11 @@
   // been removed from the active map, or was retrieved from the history DB.
   DownloadItem* GetDownloadItem(int id);
 
+  DownloadHistory* download_history() { return download_history_.get(); }

[Randy Smith (Not in Mondays), 2011/07/10 19:37:40]

I'm inclined to think that this should be const (the method, not the return
value).  

(I'm not ecstatic with making it visible at all, but I couldn't come up with
another suggestion as to how to accomplish the same thing.  But if you see some
clean way to fold the Add/Update entry for download items into download manger,
I'd prefer that.)

[achuithb, 2011/07/13 21:48:16]

I made it const (along with some other accessors), and it's only visible for the
browser test now.

+
+  // Add a DownloadItem to history_downloads_.
+  void AddDownloadItemToHistory(DownloadItem* item, int64 db_handle);
+
  private:
   // For testing.
   friend class DownloadManagerTest;
@@ -352,9 +357,6 @@
   // containers; no-op if NDEBUG.
   void AssertContainersConsistent() const;
 
-  // Add a DownloadItem to history_downloads_.
-  void AddDownloadItemToHistory(DownloadItem* item, int64 db_handle);
-
   // |downloads_| is the owning set for all downloads known to the
   // DownloadManager.  This includes downloads started by the user in
   // this session, downloads initialized from the history system, and