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

chrome/browser/ui/webui/downloads_dom_handler.cc

Index: chrome/browser/ui/webui/downloads_dom_handler.cc
diff --git a/chrome/browser/ui/webui/downloads_dom_handler.cc b/chrome/browser/ui/webui/downloads_dom_handler.cc
index 75209330acf6281ba5b485dfafc317ea080ddd0d..94f96e0353f3aa3adcf4edbc5cc9dcc4e7792b7d 100644
--- a/chrome/browser/ui/webui/downloads_dom_handler.cc
+++ b/chrome/browser/ui/webui/downloads_dom_handler.cc
@@ -146,6 +146,13 @@ void DownloadsDOMHandler::HandleGetDownloads(const ListValue* args) {
   } else {
     SendCurrentDownloads();
   }
+
+  for (OrderedDownloads::iterator it = download_items_.begin();
+       it != download_items_.end(); ++it) {
+    if ((*it)->IsComplete() && (*it)->is_path_exists())
+      (*it)->AddObserver(this);

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

This AddObserver logic getting scattered across even more methods can quickly
get harder to follow. How about just observing all downloads? Have you tried
that?

+  }
+  download_manager_->CheckExistingPaths();
 }
 
 void DownloadsDOMHandler::HandleOpenFile(const ListValue* args) {