Handle the case where IAttachmentExecute::Save() deletes a downloaded file.

content/browser/download/download_item_impl_unittest.cc

Index: content/browser/download/download_item_impl_unittest.cc
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc
index b1fe8832337c1ef3557f4ff35f127de4779e11ca..89ab387e09a33c7f7854e5455db18b9b758b4bca 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -83,16 +83,13 @@ ACTION_P(ScheduleRenameCallback, new_path) {
 
 // Schedules a task to invoke the input closure on
 // the UI thread. Should only be used as the action for
-// MockDownloadFile::Detach/Cancel as follows:
+// MockDownloadFile::Detach as follows:
 //   EXPECT_CALL(download_file, Detach(_))
-//       .WillOnce(ScheduleClosure()));
-ACTION(ScheduleClosure) {
-  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, arg0);
-}
-
-// Similarly for scheduling a completion callback.
-ACTION(ScheduleCompleteCallback) {
-  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(arg1));
+//       .WillOnce(ScheduleDetachCallback()));
+ACTION(ScheduleDetachCallback) {
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(arg0, content::DOWNLOAD_INTERRUPT_REASON_NONE));
 }
 
 }  // namespace
@@ -517,7 +514,7 @@ TEST_F(DownloadItemTest, CallbackAfterRename) {
   EXPECT_CALL(*mock_delegate(), ShouldOpenDownload(item))
       .WillOnce(Return(true));
   EXPECT_CALL(*download_file, Detach(_))
-      .WillOnce(ScheduleClosure());
+      .WillOnce(ScheduleDetachCallback());
   item->SetIsPersisted();
   item->MaybeCompleteDownload();
   RunAllPendingInMessageLoops();