Use previous target path when resuming a download after crash

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 <utility>
 
 #include "base/bind.h"
@@ skipping 202 matching lines @@
 }
 
 void DownloadManagerImpl::DetermineDownloadTarget(
     DownloadItemImpl* item, const DownloadTargetCallback& callback) {
   // Note that this next call relies on
   // DownloadItemImplDelegate::DownloadTargetCallback and
   // DownloadManagerDelegate::DownloadTargetCallback having the same
   // type.  If the types ever diverge, gasket code will need to
   // be written here.
   if (!delegate_ || !delegate_->DetermineDownloadTarget(item, callback)) {
-    base::FilePath target_path = item->GetForcedFilePath();
-    // TODO(asanka): Determine a useful path if |target_path| is empty.
+    base::FilePath target_path = item->GetForcedFilePath().empty() ?

[asanka, 2017/01/10 22:24:03]

Ditto. No need to change //content filename determination behavior. This is just
here as a fallback in case the embedder doesn't provide an implementation.

[qinmin, 2017/01/10 23:04:42]

reverted

+        item->GetTargetFilePath() : item->GetForcedFilePath();
+    // TODO(qinmin): Determine a useful path if |target_path| is empty.
     callback.Run(target_path,
                  DownloadItem::TARGET_DISPOSITION_OVERWRITE,
                  DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS,
                  target_path);
   }
 }
 
 bool DownloadManagerImpl::ShouldCompleteDownload(
     DownloadItemImpl* item, const base::Closure& complete_callback) {
   if (!delegate_ ||
@@ skipping 529 matching lines @@
   if (delegate_)
     delegate_->OpenDownload(download);
 }
 
 void DownloadManagerImpl::ShowDownloadInShell(DownloadItemImpl* download) {
   if (delegate_)
     delegate_->ShowDownloadInShell(download);
 }
 
 }  // namespace content