Save Page As MHTML

content/public/browser/download_manager_delegate.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_DELEGATE_H_
 #define CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_DELEGATE_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/file_path.h"
+#include "base/logging.h"
 #include "base/time.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/save_page_type.h"
 
 namespace content {
 
 class DownloadId;
 class DownloadItem;
 class WebContents;
 
-typedef base::Callback<void(const FilePath&, content::SavePageType)>
+// Called by SavePackage when it creates a DownloadItem.
+typedef base::Callback<void(DownloadItem*)>
+    SaveFileDownloadCreatedCallback;

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

So I had a heck of a time figuring out how this thing was used ("A callback
taking a callback passed through a callback?  We must go deeper." :-}) so I'd
rather not do it if there's some other way to arrange it.   If I'm reading the
code correctly (which I am *not* confident of, which is the basic problem :-})
it looks as if this callback exists:
* On Non-Cros: to disable safe browsing on this particular download.
* On Cros: To set the display name, and set it to temporary.

Why does this need to happen in a callback created by the file picker?  The file
picker should pick the file (and that's pretty much it).  Can we just put these
in a conditional, maybe ifdef'd, in SavePackage where this type of logic more
arguably belongs?  It'll save future maintainers (including possibly yourself)
headache and blood pressure med costs.

[benjhayden, 2012/04/28 19:09:03]

Actually, my bp is on the low end. My doctor suggested eating *more*
cholesterol, but maintaining even more bad code would be just as good.

SavePackage is in content, GDataDownloadObserver is in chrome.
How does adding a new method
DownloadManagerDelegate::SavePackageDownloadItemCreated(DownloadItem*) look?

+
+// Called by SavePackageFilePicker when the user picks a filename.
+typedef base::Callback<void(const FilePath&,
+                            content::SavePageType,
+                            const SaveFileDownloadCreatedCallback&)>
     SaveFilePathPickedCallback;
 
 // Browser's download manager: manages all downloads and destination view.
 class CONTENT_EXPORT DownloadManagerDelegate {
  public:
-  virtual ~DownloadManagerDelegate() {}
+  virtual ~DownloadManagerDelegate();
 
   // Lets the delegate know that the download manager is shutting down.
   virtual void Shutdown() {}
 
   // Returns a new DownloadId.
   virtual DownloadId GetNextId();
 
   // Notifies the delegate that a download is starting. The delegate can return
   // false to delay the start of the download, in which case it should call
   // DownloadManager::RestartDownload when it's ready.
@@ skipping 63 matching lines @@
   virtual void RemoveItemsFromPersistentStoreBetween(
       base::Time remove_begin,
       base::Time remove_end) {}
 
   // Retrieve the directories to save html pages and downloads to.
   virtual void GetSaveDir(WebContents* web_contents,
                           FilePath* website_save_dir,
                           FilePath* download_save_dir) {}
 
   // Asks the user for the path to save a page. The delegate calls the callback
-  // to give the answer, except on ChromeOS, where the saving is done by
-  // SavePackageFilePickerChromeOS. TODO(achuith): Move ChromeOS save
-  // functionality to SavePackage.
+  // to give the answer.
   virtual void ChooseSavePath(WebContents* web_contents,
                               const FilePath& suggested_path,
                               const FilePath::StringType& default_extension,
                               bool can_save_as_complete,
-                              SaveFilePathPickedCallback callback) {}
+                              const SaveFilePathPickedCallback& callback) {
+  }
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_DELEGATE_H_