Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(701)

Unified Diff: chrome/browser/download/download_target_determiner.cc

Issue 14625016: downloads: Make GetCrDownloadPath a static method on DownloadTargetDeterminer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/download/download_target_determiner.cc
diff --git a/chrome/browser/download/download_target_determiner.cc b/chrome/browser/download/download_target_determiner.cc
index f077cae959b53d955f24572367416098d83e08c6..caf1083c83c2e3415d404b55ec3a3fb7f0ebd39d 100644
--- a/chrome/browser/download/download_target_determiner.cc
+++ b/chrome/browser/download/download_target_determiner.cc
@@ -427,7 +427,7 @@ DownloadTargetDeterminer::Result
// Other safe downloads get a .crdownload suffix for their intermediate name.
if (danger_type_ == content::DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS) {
- intermediate_path_ = download_util::GetCrDownloadPath(local_path_);
+ intermediate_path_ = GetCrDownloadPath(local_path_);
return COMPLETE;
}
@@ -594,3 +594,10 @@ void DownloadTargetDeterminer::Start(
new DownloadTargetDeterminer(download, download_prefs,
last_selected_directory, delegate, callback);
}
+
+// static
+base::FilePath DownloadTargetDeterminer::GetCrDownloadPath(
+ const base::FilePath& suggested_path) {
+ return base::FilePath(suggested_path.value() +
+ FILE_PATH_LITERAL(".crdownload"));
+}
« no previous file with comments | « chrome/browser/download/download_target_determiner.h ('k') | chrome/browser/download/download_target_determiner_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698