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 aafab618660ffba44e9cfa8be88381a38501bcf3..8aad8eed5b5717fb398be98115a580efe94353a8 100644
--- a/chrome/browser/download/download_item.h
+++ b/chrome/browser/download/download_item.h
@@ -259,6 +259,10 @@ class DownloadItem {
   void set_open_when_complete(bool open) { open_when_complete_ = open; }
   int render_process_id() const { return render_process_id_; }
   int request_id() const { return request_id_; }
+  bool is_path_exists() const { return is_path_exists_; }

[Paweł Hajdan Jr., 2011/05/10 07:59:46]

nit: is_path_exists is a bad name. Why not just file_exists()? Here and in the
member variable.

+  void set_is_path_exists(bool is_path_exists) {

[Paweł Hajdan Jr., 2011/05/10 07:59:46]

We generally avoid raw setters like this. How about OnFileRemovalDetected that
sets the flag to false?

+    is_path_exists_ = is_path_exists;
+  }
   SafetyState safety_state() const { return safety_state_; }
   void set_safety_state(SafetyState safety_state) {
     safety_state_ = safety_state;
@@ -365,6 +369,9 @@ class DownloadItem {
   // A flag for indicating if the download should be opened at completion.
   bool open_when_complete_;
 
+  // A flag for indicating if the downloaded path exists or not.
+  bool is_path_exists_;
+
   // Whether the download is considered potentially safe or dangerous
   // (executable files are typically considered dangerous).
   SafetyState safety_state_;