Detect removed files and reflect the state in chrome://downloads and the download shelf

chrome/browser/download/download_item.h

Index: chrome/browser/download/download_item.h
diff --git a/chrome/browser/download/download_item.h b/chrome/browser/download/download_item.h
index 6aa5c8e0aabf530f5c4ecb770b2e87700e1d7add..d81c7b8853493108856a0c3c54beb805611d969e 100644
--- a/chrome/browser/download/download_item.h
+++ b/chrome/browser/download/download_item.h
@@ -135,7 +135,11 @@ class DownloadItem : public NotificationObserver {
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
-  // Returns true if it is OK to open this download.
+  // Whether it is OK to open a folder which this file is inside.

[Paweł Hajdan Jr., 2011/06/08 09:58:41]

nit: Please keep the "Returns true if" style; "whether" might be ambiguous.

[haraken1, 2011/06/08 10:35:07]

Done.

+  bool CanShowInFolder();
+
+  // Whether it is OK to register the type of this file so that
+  // it opens automatically.
   bool CanOpenDownload();
 
   // Tests if a file type should be opened automatically.
@@ -170,13 +174,16 @@ class DownloadItem : public NotificationObserver {
   // when resuming a download (assuming the server supports byte ranges).
   void Cancel(bool update_history);
 
-  // Called when all data has been saved.  Only has display effects.
-  void OnAllDataSaved(int64 size);
-
   // Called by external code (SavePackage) using the DownloadItem interface
   // to display progress when the DownloadItem should be considered complete.
   void MarkAsComplete();
 
+  // Called when all data has been saved. Only has display effects.
+  void OnAllDataSaved(int64 size);
+
+  // Called when the downloaded file is removed.
+  void OnDownloadedFileRemoved();
+
   // Download operation had an error.
   // |size| is the amount of data received so far, and |os_error| is the error
   // code that the operation received.
@@ -278,6 +285,7 @@ class DownloadItem : public NotificationObserver {
   bool is_paused() const { return is_paused_; }
   bool open_when_complete() const { return open_when_complete_; }
   void set_open_when_complete(bool open) { open_when_complete_ = open; }
+  bool file_externally_removed() const { return file_externally_removed_; }
   SafetyState safety_state() const { return safety_state_; }
   void set_safety_state(SafetyState safety_state) {
     safety_state_ = safety_state;
@@ -432,6 +440,9 @@ class DownloadItem : public NotificationObserver {
   // A flag for indicating if the download should be opened at completion.
   bool open_when_complete_;
 
+  // A flag for indicating if the downloaded file is externally removed.
+  bool file_externally_removed_;
+
   // Indicates if the download is considered potentially safe or dangerous
   // (executable files are typically considered dangerous).
   SafetyState safety_state_;