Return error to chrome.downloads.removeFile

content/public/browser/download_item.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.
 //
 // Each download is represented by a DownloadItem, and all DownloadItems
 // are owned by the DownloadManager which maintains a global list of all
 // downloads. DownloadItems are created when a user initiates a download,
 // and exist for the duration of the browser life time.
 //
 // Download observers:
@@ skipping 218 matching lines @@
   virtual const std::string& GetHash() const = 0;
 
   // Intermediate hash state, for persisting partial downloads.
   virtual const std::string& GetHashState() const = 0;
 
   // True if the file associated with the download has been removed by
   // external action.
   virtual bool GetFileExternallyRemoved() const = 0;
 
   // If the file is successfully deleted, then GetFileExternallyRemoved() will
-  // become true and DownloadItem::OnDownloadUpdated() will be called. Does
-  // nothing if GetState() == COMPLETE or GetFileExternallyRemoved() is already
-  // true.
-  virtual void DeleteFile() = 0;
+  // become true, GetFullPath() will become empty, and
+  // DownloadItem::OnDownloadUpdated() will be called. Does nothing if
+  // GetState() == COMPLETE or GetFileExternallyRemoved() is already true or
+  // GetFullPath() is already empty. The callback is always run, and it is
+  // always run asynchronously. It will be passed true if the file is
+  // successfully deleted or if GetFilePath() was already empty or if
+  // GetFileExternallyRemoved() was already true. The callback will be passed
+  // false if the DownloadItem was not yet complete or if the file could not be
+  // deleted for any reason.
+  virtual void DeleteFile(const base::Callback<void(bool)>& callback) = 0;
 
   // True if the file that will be written by the download is dangerous
   // and we will require a call to ValidateDangerousDownload() to complete.
   // False if the download is safe or that function has been called.
   virtual bool IsDangerous() const = 0;
 
   // Why |safety_state_| is not SAFE.
   virtual DownloadDangerType GetDangerType() const = 0;
 
   //    Progress State accessors -----------------------------------------------
@@ skipping 68 matching lines @@
   // return |name|. Has no effect on the final target filename.
   virtual void SetDisplayName(const base::FilePath& name) = 0;
 
   // Debug/testing -------------------------------------------------------------
   virtual std::string DebugString(bool verbose) const = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_ITEM_H_