[Downloads] Remove externally deleted items from downloads history

chrome/browser/android/download/download_manager_service.cc

Index: chrome/browser/android/download/download_manager_service.cc
diff --git a/chrome/browser/android/download/download_manager_service.cc b/chrome/browser/android/download/download_manager_service.cc
index 18eb7dd085503fc5f657d406ea21881c3990753e..2b60ad549fea79ca9781a57fe77b248ebad2ac55 100644
--- a/chrome/browser/android/download/download_manager_service.cc
+++ b/chrome/browser/android/download/download_manager_service.cc
@@ -174,7 +174,11 @@ void DownloadManagerService::CheckForExternallyRemovedDownloads(
     const JavaParamRef<jobject>& obj,
     bool is_off_the_record) {
   content::DownloadManager* manager = GetDownloadManager(is_off_the_record);
-  if (!manager)
+
+  // Once the history query is complete, download_history.cc will check for the
+  // removal of history files. If the history query is not yet complete, ignore
+  // requests to check for externally removed downloads.
+  if (!manager || !is_history_query_complete_)

[qinmin, 2016/08/24 16:30:53]

Nit: there is no need to get the manager if is_history_query_complete_ is false.

So it would be better to change this to:

if (!is_history_query_complete_) 
  return;

DownloadManger* manager = ....
....

[Theresa, 2016/08/24 22:52:09]

Done.

     return;
   manager->CheckForHistoryFilesRemoval();
 }