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

content/browser/download/download_item_impl.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_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 190 matching lines @@
  private:
   // Fine grained states of a download.
   enum DownloadInternalState {
     // Unless otherwise specified, state transitions are linear forward
     // in this list.
 
     // Includes both before and after file name determination.
     // TODO(rdsmith): Put in state variable for file name determination.
     IN_PROGRESS_INTERNAL,
 
-    // Between commit point (dispatch of download file release) and
-    // completed.  Embedder may be opening the file in this state.
-    // Note that the DownloadItem may be deleted (by shutdown) in this
-    // state.
+    // Between commit point (dispatch of download file release) and completed.
+    // Embedder may be opening the file in this state.  Note that the
+    // DownloadItem may be deleted (by shutdown) or interrupted (e.g. due to a
+    // failure during AnnotateWithSourceInformation()) in this state.
     COMPLETING_INTERNAL,
 
     // After embedder has had a chance to auto-open.  User may now open
     // or auto-open based on extension.
     COMPLETE_INTERNAL,
 
     // User has cancelled the download.
     // Only incoming transition IN_PROGRESS->
     CANCELLED_INTERNAL,
 
@@ skipping 47 matching lines @@
 
   // Called after the delegate has given the go-ahead to actually complete
   // the download.
   void ReadyForDownloadCompletionDone();
 
   void OnDownloadRenamedToFinalName(DownloadInterruptReason reason,
                                     const FilePath& full_path);
 
   void ReleaseDownloadFile();
 
-  void OnDownloadFileReleased();
+  // TODO(rdsmith,asanka): Move the AnnotateWithSourceInformation() call to the
+  //     final rename and eliminate the interrupt reason callback.
+  void OnDownloadFileReleased(DownloadInterruptReason reason);
 
   // Called when the entire download operation (including renaming etc)
   // is completed.
   void Completed();
 
   // Helper routines -----------------------------------------------------------
 
   // Cancel the DownloadFile if we have it.
   void CancelDownloadFile();
 
@@ skipping 186 matching lines @@
   const net::BoundNetLog bound_net_log_;
 
   base::WeakPtrFactory<DownloadItemImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadItemImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_H_