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

chrome/browser/download/save_package.h

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_DOWNLOAD_SAVE_PACKAGE_H_
 #define CHROME_BROWSER_DOWNLOAD_SAVE_PACKAGE_H_
 #pragma once
 
 #include <queue>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/gtest_prod_util.h"
 #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"
 
 class SaveFileManager;
 class SaveItem;
 class SavePackage;
 class DownloadItem;
 class DownloadManager;
 class GURL;
@@ skipping 127 matching lines @@
   ~SavePackage();
 
   // Notes from Init() above applies here as well.
   void InternalInit();
 
   void Stop();
   void CheckFinish();
   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);
 
   // Create a file name based on the response from the server.
   bool GenerateFileName(const std::string& disposition,
                         const GURL& url,
                         bool need_html_ext,
                         FilePath::StringType* generated_name);
 
   // Get all savable resource links from current web page, include main
@@ skipping 135 matching lines @@
 
   friend class SavePackageTest;
   FRIEND_TEST_ALL_PREFIXES(SavePackageTest, TestSuggestedSaveNames);
 
   ScopedRunnableMethodFactory<SavePackage> method_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SavePackage);
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_SAVE_PACKAGE_H_