Save Page As MHTML

chrome/browser/download/save_package_file_picker.h

Index: chrome/browser/download/save_package_file_picker.h
diff --git a/chrome/browser/download/save_package_file_picker.h b/chrome/browser/download/save_package_file_picker.h
index c3b622c220e1c3b86cf6a28bba46c4a6ff1634c8..e74fa5427bb8db8ffc2a047fe803cffc44adef89 100644
--- a/chrome/browser/download/save_package_file_picker.h
+++ b/chrome/browser/download/save_package_file_picker.h
@@ -6,7 +6,6 @@
 #define CHROME_BROWSER_DOWNLOAD_SAVE_PACKAGE_FILE_PICKER_H_
 #pragma once
 
-#include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/ui/select_file_dialog.h"
 #include "content/public/browser/download_manager_delegate.h"
@@ -17,8 +16,8 @@ class DownloadPrefs;
 class SavePackageFilePicker : public SelectFileDialog::Listener {
  public:
   SavePackageFilePicker(content::WebContents* web_contents,
-                        const FilePath& suggested_path,
-                        const FilePath::StringType& default_extension,
+                        FilePath suggested_path,

[achuithb, 2012/04/24 22:06:09]

I don't think you should do this. If you need to modify this value, I'd prefer
creating a local temporary instead.

[achuithb, 2012/04/24 22:28:14]

Just saw Randy's note. Personally not a fan of function arguments that are not
either const references (in params), or pointers (out params), but this is fine.
Sorry for the trouble.

+                        FilePath::StringType default_extension,
                         bool can_save_as_complete,
                         DownloadPrefs* download_prefs,
                         content::SaveFilePathPickedCallback callback);
@@ -32,12 +31,17 @@ class SavePackageFilePicker : public SelectFileDialog::Listener {
   // SelectFileDialog::Listener implementation.
   virtual void FileSelected(const FilePath& path,
                             int index,
-                            void* params) OVERRIDE;
-  virtual void FileSelectionCanceled(void* params) OVERRIDE;
+                            void* unused_params) OVERRIDE;
+  virtual void FileSelectionCanceled(void* unused_params) OVERRIDE;
+
+  bool ShouldSaveAsMHTML() const;
 
   // Used to look up the renderer process for this request to get the context.
   int render_process_id_;
 
+  // Whether the web page can be saved as a complete HTML file.
+  bool can_save_as_complete_;
+
   content::SaveFilePathPickedCallback callback_;
 
   // For managing select file dialogs.