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..aecc3c3d380b34d3b0ec9d91bdd1515eae903ecc 100644
--- a/content/browser/download/save_package.h
+++ b/content/browser/download/save_package.h
@@ -35,12 +35,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
@@ -118,6 +120,9 @@ class CONTENT_EXPORT SavePackage
  private:
   friend class base::RefCountedThreadSafe<SavePackage>;
 
+  // Callback for content::WebContents::GenerateMHTML().
+  void MHTMLGenerated(const FilePath& path, int64 size);

[cbentzel, 2012/04/17 14:45:34]

Perhaps prefix with On to make it clear that this is a callback handler. 

[benjhayden, 2012/04/17 17:14:36]

Done.

+
   // For testing only.
   SavePackage(content::WebContents* web_contents,
               const FilePath& file_full_path,