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 9fa5d87d18e7b07a68c1627ca73506b6f505af44..d3ef79273428bf7a9bdc36e208168e0f7711893e 100644
--- a/content/browser/download/save_package.h
+++ b/content/browser/download/save_package.h
@@ -35,12 +35,15 @@ 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 complete-html-single-file and providing the information for

[Randy Smith (Not in Mondays), 2012/04/16 17:45:47]

nit, suggestion: I think of the target audience for comments as being other
developers, and I think of other developers as having an easier time with a save
format description of MHTML than with complete-html-single-file.  And there's
also the change in enum.  Would it make sense to change this to refer to MHTML?

[benjhayden, 2012/04/16 19:08:06]

Done.

+// 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 complete-html-single-file 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 +121,9 @@ class CONTENT_EXPORT SavePackage
  private:
   friend class base::RefCountedThreadSafe<SavePackage>;
 
+  // Callback for content::WebContents::GenerateMHTML().
+  void MHTMLGenerated(const FilePath& path, int64 size);
+
   // For testing only.
   SavePackage(content::WebContents* web_contents,
               const FilePath& file_full_path,