Modified cancel and interrupt processing to avoid race with history.

chrome/browser/download/download_history.cc

Index: chrome/browser/download/download_history.cc
diff --git a/chrome/browser/download/download_history.cc b/chrome/browser/download/download_history.cc
index 115541a4d24742ed52fa9ed9683f915109b60175..fc400309399b37db0630904a282ad9285b58c43c 100644
--- a/chrome/browser/download/download_history.cc
+++ b/chrome/browser/download/download_history.cc
@@ -120,14 +120,14 @@ void DownloadHistory::UpdateDownloadPath(DownloadItem* download_item,
     hs->UpdateDownloadPath(new_path, download_item->db_handle());
 }
 
-void DownloadHistory::RemoveEntry(DownloadItem* download_item) {
+void DownloadHistory::RemoveEntry(int64 db_handle) {
   // No update necessary if the download was initiated while in incognito mode.
-  if (download_item->db_handle() <= kUninitializedHandle)
+  if (db_handle <= kUninitializedHandle)

[benjhayden, 2011/07/06 18:12:06]

Do you want to start using IsFakeHandle(db_handle) everywhere?
My browser pref id work will obviate GetNextFakeDbHandle, so it's up to you.

[Randy Smith (Not in Mondays), 2011/07/07 22:02:23]

I think of IsFakeHandle as an interface to DownloadHistory, so I'm not worried
about using it inside DownloadHistory.  It's so the code outside of
DownloadHistory don't have to know about how fake DB handles are created.

Having said that, it appears I've created this without using it (probably cruft
from an intermediate commit) so I'm going to remove it.

     return;
 
   HistoryService* hs = profile_->GetHistoryService(Profile::EXPLICIT_ACCESS);
   if (hs)
-    hs->RemoveDownload(download_item->db_handle());
+    hs->RemoveDownload(db_handle);
 }
 
 void DownloadHistory::RemoveEntriesBetween(const base::Time remove_begin,
@@ -141,6 +141,11 @@ int64 DownloadHistory::GetNextFakeDbHandle() {
   return next_fake_db_handle_--;
 }
 
+// static
+bool DownloadHistory::IsFakeHandle(int64 db_handle) {
+  return db_handle <= 0;

[benjhayden, 2011/07/06 18:12:06]

db_handle <= kUninitializedHandle;

[Randy Smith (Not in Mondays), 2011/07/07 22:02:23]

Done.

+}
+
 void DownloadHistory::OnGotVisitCountToHost(HistoryService::Handle handle,
                                             bool found_visits,
                                             int count,