Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(463)

Unified Diff: content/browser/download/save_package.h

Issue 10069014: Save Page As MHTML (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cros Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..61587e3ed927200a3d7fe6c49499bd6ccf517be3 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 OnMHTMLGenerated(const FilePath& path, int64 size);
+
// For testing only.
SavePackage(content::WebContents* web_contents,
const FilePath& file_full_path,

Powered by Google App Engine
This is Rietveld 408576698