Save Page As MHTML

chrome/browser/download/chrome_download_manager_delegate.h

Index: chrome/browser/download/chrome_download_manager_delegate.h
diff --git a/chrome/browser/download/chrome_download_manager_delegate.h b/chrome/browser/download/chrome_download_manager_delegate.h
index e5ee17dffb732aa0f92b0d3a75591949954ce948..76346925c7a879f90fbe67dd55fab5ffefd12362 100644
--- a/chrome/browser/download/chrome_download_manager_delegate.h
+++ b/chrome/browser/download/chrome_download_manager_delegate.h
@@ -83,11 +83,15 @@ class ChromeDownloadManagerDelegate
       const FilePath& suggested_path,
       const FilePath::StringType& default_extension,
       bool can_save_as_complete,
-      content::SaveFilePathPickedCallback callback) OVERRIDE;
+      const content::SaveFilePathPickedCallback& callback) OVERRIDE;
 
   DownloadPrefs* download_prefs() { return download_prefs_.get(); }
   DownloadHistory* download_history() { return download_history_.get(); }
 
+#if defined(ENABLE_SAFE_BROWSING)
+  static void DisableSafeBrowsing(content::DownloadItem* item);
+#endif

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

It looks like CDMD has to date only used ifdefs to null out subsections of code
rather than whole function APIs.  If Noe' et al have an opinion, it should be
followed, but in the absence of this are you willing to declare this function
but make its behavior null if Safe browsing isn't defined, similar to
GetDownloadProtectionService?

[benjhayden, 2012/04/30 13:37:19]

Done.

+
  protected:
   // So that test classes can inherit from this for override purposes.
   virtual ~ChromeDownloadManagerDelegate();