Index: content/browser/download/download_item_impl.cc |
diff --git a/content/browser/download/download_item_impl.cc b/content/browser/download/download_item_impl.cc |
index e2bf57a2c0cbc6dde6e0b190f61c8a87db147ed6..2f3a7a093d43fe93177c8c4abf2efcd40f34d2e6 100644 |
--- a/content/browser/download/download_item_impl.cc |
+++ b/content/browser/download/download_item_impl.cc |
@@ -1068,18 +1068,6 @@ void DownloadItemImpl::OnDownloadFileInitialized( |
return; |
} |
- // If we're resuming an interrupted download, we may already know the download |
- // target so we can skip target name determination. GetFullPath() is non-empty |
- // for interrupted downloads where the intermediate file is still present, and |
- // also for downloads with forced paths. |
- if (!GetTargetFilePath().empty() && !GetFullPath().empty()) { |
- // TODO(rdsmith/asanka): Check to confirm that the target path isn't |
- // present on disk; if it is, we should re-do filename determination to |
- // give the user a chance not to collide. |
- MaybeCompleteDownload(); |
- return; |
- } |
- |
delegate_->DetermineDownloadTarget( |
this, base::Bind(&DownloadItemImpl::OnDownloadTargetDetermined, |
weak_ptr_factory_.GetWeakPtr())); |
@@ -1122,6 +1110,15 @@ void DownloadItemImpl::OnDownloadTargetDetermined( |
// space/permission/availability constraints. |
DCHECK(intermediate_path.DirName() == target_path.DirName()); |
+ // During resumption, we may choose to proceed with the same intermediate |
+ // file. No rename is necessary if our intermediate file already has the |
+ // correct name. |
+ if (intermediate_path == current_path_) { |
+ OnDownloadRenamedToIntermediateName(DOWNLOAD_INTERRUPT_REASON_NONE, |
+ intermediate_path); |
+ return; |
+ } |
+ |
// Rename to intermediate name. |
// TODO(asanka): Skip this rename if AllDataSaved() is true. This avoids a |
// spurious rename when we can just rename to the final |