Save Page As MHTML

chrome/browser/download/save_page_browsertest.cc

Index: chrome/browser/download/save_page_browsertest.cc
diff --git a/chrome/browser/download/save_page_browsertest.cc b/chrome/browser/download/save_page_browsertest.cc
index 3b0cfcaf7390baa3b23863228d1d701bfa6bd2f6..9281c90369c3842ffb4e46b95abd7780cc6440ac 100644
--- a/chrome/browser/download/save_page_browsertest.cc
+++ b/chrome/browser/download/save_page_browsertest.cc
@@ -4,6 +4,7 @@
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
@@ -17,10 +18,14 @@
 #include "chrome/browser/download/download_service_factory.h"
 #include "chrome/browser/download/save_package_file_picker.h"
 #include "chrome/browser/net/url_request_mock_util.h"
+#include "chrome/browser/prefs/pref_member.h"
+#include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/common/chrome_paths.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
@@ -53,6 +58,10 @@ static const char* kAppendedExtension =
 }  // namespace
 
 class SavePageBrowserTest : public InProcessBrowserTest {
+ public:
+  SavePageBrowserTest() {}
+  virtual ~SavePageBrowserTest();
+
  protected:
   void SetUp() OVERRIDE {
     ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_dir_));
@@ -61,6 +70,8 @@ class SavePageBrowserTest : public InProcessBrowserTest {
   }
 
   void SetUpOnMainThread() OVERRIDE {
+    browser()->profile()->GetPrefs()->SetFilePath(
+        prefs::kDownloadDefaultDirectory, save_dir_.path());
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(&chrome_browser_net::SetUrlRequestMocksEnabled, true));
@@ -154,12 +165,27 @@ class SavePageBrowserTest : public InProcessBrowserTest {
 
   void CheckDownloadHistory(const GURL& url,
                             const FilePath& path,
-                            int64 num_files_) {
+                            int64 num_files) {
     QueryDownloadHistory();
 
-    EXPECT_NE(std::find_if(history_entries_.begin(), history_entries_.end(),
-        DownloadPersistentStoreInfoMatch(url, path, num_files_)),
-        history_entries_.end());
+    std::vector<DownloadPersistentStoreInfo>::iterator found =
+      std::find_if(history_entries_.begin(), history_entries_.end(),
+                   DownloadPersistentStoreInfoMatch(url, path, num_files));
+
+    if (found == history_entries_.end()) {
+      LOG(ERROR) << "Missing url=" << url.spec()
+                 << " path=" << path.value()
+                 << " received=" << num_files;
+      for (size_t index = 0; index < history_entries_.size(); ++index) {
+        LOG(ERROR) << "History@" << index << ": url="
+                   << history_entries_[index].url.spec()
+                   << " path=" << history_entries_[index].path.value()
+                   << " received=" << history_entries_[index].received_bytes
+                   << " total=" << history_entries_[index].total_bytes
+                   << " state=" << history_entries_[index].state;
+      }
+      EXPECT_TRUE(false);
+    }
   }
 
   std::vector<DownloadPersistentStoreInfo> history_entries_;
@@ -169,8 +195,14 @@ class SavePageBrowserTest : public InProcessBrowserTest {
 
   // Temporary directory we will save pages to.
   ScopedTempDir save_dir_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(SavePageBrowserTest);
 };
 
+SavePageBrowserTest::~SavePageBrowserTest() {
+}
+
 IN_PROC_BROWSER_TEST_F(SavePageBrowserTest, SaveHTMLOnly) {
   GURL url = NavigateToMockURL("a");
 
@@ -335,3 +367,40 @@ IN_PROC_BROWSER_TEST_F(SavePageBrowserTest, CleanFilenameFromPageTitle) {
   EXPECT_TRUE(file_util::DieFileDie(dir, true));
 }
 #endif
+
+#if !defined(OS_CHROMEOS)

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

Why does this not work on chromeos?

[benjhayden, 2012/04/26 15:05:57]

Done.

+class SavePageAsMHTMLBrowserTest : public SavePageBrowserTest {
+ public:
+  SavePageAsMHTMLBrowserTest() {}
+  virtual ~SavePageAsMHTMLBrowserTest();
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    command_line->AppendSwitch(switches::kSavePageAsMHTML);
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(SavePageAsMHTMLBrowserTest);
+};
+
+SavePageAsMHTMLBrowserTest::~SavePageAsMHTMLBrowserTest() {
+}
+
+IN_PROC_BROWSER_TEST_F(SavePageAsMHTMLBrowserTest, SavePageAsMHTML) {
+  GURL url = NavigateToMockURL("b");
+  FilePath download_dir = DownloadPrefs::FromDownloadManager(
+      GetDownloadManager())->download_path();
+  FilePath full_file_name = download_dir.AppendASCII(std::string(
+      "Test page for saving page feature.mhtml"));
+  SavePackageFilePicker::SetShouldPromptUser(false);
+  ui_test_utils::WindowedNotificationObserver observer(
+        content::NOTIFICATION_SAVE_PACKAGE_SUCCESSFULLY_FINISHED,
+        content::NotificationService::AllSources());
+  browser()->SavePage();
+  observer.Wait();
+  CheckDownloadHistory(url, full_file_name, 1);
+
+  EXPECT_TRUE(file_util::PathExists(full_file_name));
+  int64 actual_file_size = -1;
+  EXPECT_TRUE(file_util::GetFileSize(full_file_name, &actual_file_size));
+  EXPECT_EQ(2760, actual_file_size);
+}
+#endif