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

Unified Diff: content/browser/download/save_package.cc

Issue 1410333006: Enough hacks to make wstring printfs unneeded (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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: content/browser/download/save_package.cc
diff --git a/content/browser/download/save_package.cc b/content/browser/download/save_package.cc
index 96871950bac431327eb405ebbbcbef4711ef8d96..4528e60bce4641eb88a251730778454008980c8e 100644
--- a/content/browser/download/save_package.cc
+++ b/content/browser/download/save_package.cc
@@ -527,8 +527,8 @@ bool SavePackage::GenerateFileName(const std::string& disposition,
return false;
} else {
for (int i = ordinal_number; i < kMaxFileOrdinalNumber; ++i) {
- base::FilePath::StringType new_name = base_file_name +
- base::StringPrintf(FILE_PATH_LITERAL("(%d)"), i) + file_name_ext;
+ base::FilePath::StringType new_name = base_file_name/* +
+ base::StringPrintf(FILE_PATH_LITERAL("(%d)"), i) + file_name_ext*/;
if (file_name_set_.find(new_name) == file_name_set_.end()) {
// Resolved name conflict.
file_name = new_name;

Powered by Google App Engine
This is Rietveld 408576698