Modified cancel and interrupt processing to avoid race with history.

chrome/browser/automation/automation_provider_observers.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/automation/automation_provider_observers.h"
 
 #include <deque>
 #include <string>
 #include <vector>
 
@@ skipping 1439 matching lines @@
 AutomationProviderDownloadItemObserver::
     ~AutomationProviderDownloadItemObserver() {}
 
 void AutomationProviderDownloadItemObserver::OnDownloadUpdated(
     DownloadItem* download) {
   interrupted_ |= download->IsInterrupted();
   // If any download was interrupted, on the next update each outstanding
   // download is cancelled.
   if (interrupted_) {
     // |Cancel()| does nothing if |download| is already interrupted.
-    download->Cancel(true);
+    download->Cancel();
     RemoveAndCleanupOnLastEntry(download);
   }
 
   if (download->IsComplete())
     RemoveAndCleanupOnLastEntry(download);
 }
 
 // We don't want to send multiple messages, as the behavior is undefined.
 // Set |interrupted_| on error, and on the last download completed/
 // interrupted, send either an error or a success message.
@@ skipping 1116 matching lines @@
 void DragTargetDropAckNotificationObserver::Observe(
     NotificationType type,
     const NotificationSource& source,
     const NotificationDetails& details) {
   if (automation_) {
     AutomationJSONReply(automation_,
                         reply_message_.release()).SendSuccess(NULL);
   }
   delete this;
 }