Add files saved using 'Save page as' to the download history....

chrome/browser/download/save_package.h

Index: chrome/browser/download/save_package.h
===================================================================
--- chrome/browser/download/save_package.h	(revision 73953)
+++ chrome/browser/download/save_package.h	(working copy)
@@ -16,6 +16,7 @@
 #include "base/hash_tables.h"
 #include "base/ref_counted.h"
 #include "base/task.h"
+#include "chrome/browser/history/download_create_info.h"

[Paweł Hajdan Jr., 2011/02/11 19:17:49]

nit: Is this include really needed? It should be possible to forward-declare
DownloadCreateInfo.

[magnus, 2011/02/13 03:32:41]

See comment below.

 #include "chrome/browser/shell_dialogs.h"
 #include "chrome/browser/tab_contents/tab_contents_observer.h"
 #include "googleurl/src/gurl.h"
@@ -163,6 +164,11 @@
   void SaveNextFile(bool process_all_remainder_items);
   void DoSavingProcess();
 
+  // Called when Save Page As entry commited to the history system

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

nit: "is commited"

[magnus, 2011/02/13 03:32:41]

Done.

+  void OnDownloadEntryAdded(DownloadCreateInfo info, int64 db_handle);

[Paweł Hajdan Jr., 2011/02/11 19:17:49]

nit: |info| should probably be passed as const DownloadCreateInfo& info.

[magnus, 2011/02/13 03:32:41]

This is actually a callback function for the history system, which expect
DownloadCreateInfo to be passed by value. 

+  // Called when a Save Page As download is started.

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

nit: I'd put a blank line above this comment.

[magnus, 2011/02/13 03:32:41]

Done.

+  void CreateDownloadItem(const FilePath& path, const GURL& url, bool is_otr);
+
   // TabContentsObserver implementation.
   virtual bool OnMessageReceived(const IPC::Message& message);