Save Page As MHTML

content/browser/download/save_package.h

Index: content/browser/download/save_package.h
diff --git a/content/browser/download/save_package.h b/content/browser/download/save_package.h
index 8fefebd7ed5ae2d1a06ea037644100afc972c56e..05908a04f46b2d0ef5a70b64b7fe6a4232f6e39d 100644
--- a/content/browser/download/save_package.h
+++ b/content/browser/download/save_package.h
@@ -19,6 +19,7 @@
 #include "base/time.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/download_item.h"
+#include "content/public/browser/download_manager_delegate.h"
 #include "content/public/browser/save_page_type.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "googleurl/src/gurl.h"
@@ -35,12 +36,14 @@ class WebContents;
 }
 
 // The SavePackage object manages the process of saving a page as only-html or
-// complete-html and providing the information for displaying saving status.
-// Saving page as only-html means means that we save web page to a single HTML
-// file regardless internal sub resources and sub frames.
-// Saving page as complete-html page means we save not only the main html file
-// the user told it to save but also a directory for the auxiliary files such
-// as all sub-frame html files, image files, css files and js files.
+// complete-html or MHTML and providing the information for displaying saving
+// status.  Saving page as only-html means means that we save web page to a
+// single HTML file regardless internal sub resources and sub frames.  Saving
+// page as complete-html page means we save not only the main html file the user
+// told it to save but also a directory for the auxiliary files such as all
+// sub-frame html files, image files, css files and js files.  Saving page as
+// MHTML means the same thing as complete-html, but it uses the MHTML format to
+// contain the html and all auxiliary files in a single text file.
 //
 // Each page saving job may include one or multiple files which need to be
 // saved. Each file is represented by a SaveItem, and all SaveItems are owned
@@ -88,7 +91,9 @@ class CONTENT_EXPORT SavePackage
   // Need to make sure that this method must be called in the UI thread because
   // using g_browser_process on a non-UI thread can cause crashes during
   // shutdown.
-  bool Init();
+  // |download_created_callback| will be called when the DownloadItem is

[Randy Smith (Not in Mondays), 2012/04/27 18:24:04]

nit: There is no entity named download_created_callback in the below
declaration.

[benjhayden, 2012/04/30 13:37:19]

Done.

+  // created, before data is written to disk.
+  bool Init(const content::SaveFileDownloadCreatedCallback& cb);
 
   // Cancel all in progress request, might be called by user or internal error.
   void Cancel(bool user_action);
@@ -118,6 +123,9 @@ class CONTENT_EXPORT SavePackage
  private:
   friend class base::RefCountedThreadSafe<SavePackage>;
 
+  // Callback for content::WebContents::GenerateMHTML().
+  void OnMHTMLGenerated(const FilePath& path, int64 size);
+
   // For testing only.
   SavePackage(content::WebContents* web_contents,
               const FilePath& file_full_path,
@@ -185,7 +193,10 @@ class CONTENT_EXPORT SavePackage
                                    const std::string& accept_langs);
   void ContinueGetSaveInfo(const FilePath& suggested_path,
                            bool can_save_as_complete);
-  void OnPathPicked(const FilePath& final_name, content::SavePageType type);
+  void OnPathPicked(
+      const FilePath& final_name,
+      content::SavePageType type,
+      const content::SaveFileDownloadCreatedCallback& cb);
   void OnReceivedSavableResourceLinksForCurrentPage(
       const std::vector<GURL>& resources_list,
       const std::vector<GURL>& referrers_list,