Test file errors in downloads.

chrome/browser/download/download_browsertest.cc

 // 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.
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 37 matching lines @@
 #include "content/browser/net/url_request_mock_http_job.h"
 #include "content/browser/net/url_request_slow_download_job.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/renderer_host/resource_dispatcher_host.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/download_manager.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/context_menu_params.h"
 #include "content/public/common/page_transition_types.h"
+#include "content/test/test_file_error_injector.h"
 #include "content/test/test_navigation_observer.h"
 #include "net/base/net_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserThread;
 using content::DownloadItem;
 using content::DownloadManager;
 using content::WebContents;
 
 namespace {
@@ skipping 231 matching lines @@
   };
 
   // Information passed in to |DownloadFileCheckErrors()|.
   struct DownloadInfo {
     const char* url_name;  // URL for the download.
     DownloadMethod download_method;  // Navigation or Direct.
     InterruptReason reason;  // Download interrupt reason (NONE is OK).
     bool show_download_item;  // True if the download item appears on the shelf.
   };
 
+  struct FileErrorInjectInfo {
+    DownloadInfo download_info;  // See above.
+    content::FileOperationCode code;  // A code indicating what file operation
+                                      // to affect.
+    int operation_index;  // The number of the operation to affect (0 = first).
+    net::Error error_to_inject;  // The net error to inject into the file op.
+  };
+
   DownloadTest() {
     EnableDOMAutomation();
   }
 
   void SetUpOnMainThread() OVERRIDE {
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(&chrome_browser_net::SetUrlRequestMocksEnabled, true));
   }
 
@@ skipping 410 matching lines @@
     ASSERT_EQ(item_count, download_items.size());
 
     if (download_info.show_download_item) {
       DownloadItem* item = download_items[0];
       ASSERT_EQ(url, item->GetOriginalUrl());
 
       ASSERT_EQ(download_info.reason, item->GetLastReason());
     }
   }
 
+  void DownloadInsertFileErrorCheckErrors(const FileErrorInjectInfo& info) {
+    // Do initial setup.
+    ASSERT_TRUE(InitialSetup(false));
+
+    // Set up file failures.
+    // |injector| will be owned by the DownloadFileManager (indirectly).

[cbentzel, 2012/02/27 02:18:54]

How hard would it be to change this so the injector is owned by the caller of
Create, and the DownloadFileManager etc. could use a weak pointer?

May be tough with threads. But worth considering.

[ahendrickson, 2012/02/28 03:28:56]

Switched to using RefCountedThreadSafe for the injector.

+    content::TestFileErrorInjector* injector =
+        content::TestFileErrorInjector::Create();
+    injector->InjectError(0,  // First DownloadFile created.

[cbentzel, 2012/02/27 02:18:54]

I'd remove this argument for now.

[ahendrickson, 2012/02/28 03:28:56]

How strongly do you feel about that?

[cbentzel, 2012/02/28 10:57:29]

Pretty strongly. I don't like adding arguments that are unneeded/unused: it adds
complexity to the API, to callers, and to implementation.

+                          info.code,
+                          info.operation_index,

[cbentzel, 2012/02/27 02:18:54]

Do you need operation_index? It looks like it's always set to 0.

[ahendrickson, 2012/02/28 03:28:56]

Right now the unit tests only try to download small files, which result in
having only one write operation.  I'd like to have tests in the future that use
larger files, and have an error in, say, the 5th operation.

[cbentzel, 2012/02/28 10:57:29]

Again, I'd hold off until you have concrete need for this.

+                          info.error_to_inject);
+
+    DownloadFileCheckErrors(info.download_info);
+
+    EXPECT_EQ(1u, injector->FileCreationCount());
+    EXPECT_EQ(0u, injector->CurrentFileCount());
+  }
+
  private:
   // Location of the test data.
   FilePath test_dir_;
 
   // Location of the downloads directory for these tests
   ScopedTempDir downloads_directory_;
 };
 
 // NOTES:
 //
@@ skipping 1325 matching lines @@
     "zip_file_not_found.zip",
     DOWNLOAD_NAVIGATE,
     DOWNLOAD_INTERRUPT_REASON_SERVER_FAILED,
     false
   };
 
   // Do initial setup.
   ASSERT_TRUE(InitialSetup(false));
   DownloadFileCheckErrors(download_info);
 }
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNoSpaceOpenNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NO_SPACE,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FILE_NO_SPACE

[cbentzel, 2012/02/27 02:18:54]

Why would we get NO_SPACE on INITIALIZE?

[ahendrickson, 2012/02/28 03:28:56]

Wouldn't we get that if there are no bytes available on a disk?

+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNoSpaceOpenDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NO_SPACE,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FILE_NO_SPACE
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNoSpaceWriteNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NO_SPACE,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_FILE_NO_SPACE
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNoSpaceWriteDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NO_SPACE,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_FILE_NO_SPACE
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileFailedOpenNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FAILED
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileFailedOpenDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FAILED
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileFailedWriteNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_FAILED
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileFailedWriteDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_FAILED
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNameTooLongOpenNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NAME_TOO_LONG,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FILE_PATH_TOO_LONG
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadErrorFileNameTooLongOpenDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_NAME_TOO_LONG,
+      1
+    },
+    content::FILE_OPERATION_INITIALIZE,
+    0,
+    net::ERR_FILE_PATH_TOO_LONG
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest,
+                       DownloadErrorFileInvalidHandleWriteNavigate) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_NAVIGATE,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_INVALID_HANDLE
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest,
+                       DownloadErrorFileInvalidHandleWriteDirect) {
+  FileErrorInjectInfo error_info = {
+    { "a_zip_file.zip",
+      DOWNLOAD_DIRECT,
+      DOWNLOAD_INTERRUPT_REASON_FILE_FAILED,
+      1
+    },
+    content::FILE_OPERATION_WRITE,
+    0,
+    net::ERR_INVALID_HANDLE
+  };
+
+  DownloadInsertFileErrorCheckErrors(error_info);
+}