Pull delegate check back into IsDownloadReadyForCompletion so that if

content/browser/download/download_manager_impl.cc

 // Copyright (c) 2012 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 "content/browser/download/download_manager_impl.h"
 
 #include <iterator>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 219 matching lines @@
   if (!delegate_ || !delegate_->DetermineDownloadTarget(item, callback)) {
     FilePath target_path = item->GetForcedFilePath();
     // TODO(asanka): Determine a useful path if |target_path| is empty.
     callback.Run(target_path,
                  DownloadItem::TARGET_DISPOSITION_OVERWRITE,
                  DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS,
                  target_path);
   }
 }
 
-void DownloadManagerImpl::ReadyForDownloadCompletion(
+bool DownloadManagerImpl::ShouldCompleteDownload(
     DownloadItemImpl* item, const base::Closure& complete_callback) {
   if (!delegate_ ||
       delegate_->ShouldCompleteDownload(item, complete_callback)) {
-    complete_callback.Run();
+    return true;
   }
   // Otherwise, the delegate has accepted responsibility to run the
   // callback when the download is ready for completion.
+  return false;
 }
 
 bool DownloadManagerImpl::ShouldOpenFileBasedOnExtension(const FilePath& path) {
   if (!delegate_)
     return false;
 
   return delegate_->ShouldOpenFileBasedOnExtension(path);
 }
 
 bool DownloadManagerImpl::ShouldOpenDownload(
@@ skipping 369 matching lines @@
        it != downloads_.end(); ++it) {
     DownloadItemImpl* item = it->second;
     if (item->IsComplete() &&
         !item->GetOpened())
       ++num_unopened;
   }
   RecordOpensOutstanding(num_unopened);
 }
 
 }  // namespace content