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

Unified Diff: chrome/installer/setup/install.cc

Issue 902643002: Revert "Enable positional parameters for base::vsnprintf and base::vswprintf on Windows." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixes Created 5 years, 10 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/installer/setup/install.cc
diff --git a/chrome/installer/setup/install.cc b/chrome/installer/setup/install.cc
index f4c091b0c50b8d686830447091ffb3755fb6e6c7..55e6300fe2130b377881a772efb422f03d2770b7 100644
--- a/chrome/installer/setup/install.cc
+++ b/chrome/installer/setup/install.cc
@@ -302,20 +302,20 @@ bool CreateVisualElementsManifest(const base::FilePath& src_path,
<< installer::kVisualElementsManifest << " to " << src_path.value();
return true;
} else {
- // A printf_p-style format string for generating the visual elements
+ // A printf-style format string for generating the visual elements
// manifest. Required arguments, in order, are:
// - Localized display name for the product.
- // - Relative path to the VisualElements directory.
+ // - Relative path to the VisualElements directory, three times.
static const char kManifestTemplate[] =
"<Application>\r\n"
" <VisualElements\r\n"
- " DisplayName='%1$ls'\r\n"
- " Logo='%2$ls\\Logo.png'\r\n"
- " SmallLogo='%2$ls\\SmallLogo.png'\r\n"
+ " DisplayName='%ls'\r\n"
+ " Logo='%ls\\Logo.png'\r\n"
+ " SmallLogo='%ls\\SmallLogo.png'\r\n"
" ForegroundText='light'\r\n"
" BackgroundColor='#323232'>\r\n"
" <DefaultTile ShowName='allLogos'/>\r\n"
- " <SplashScreen Image='%2$ls\\splash-620x300.png'/>\r\n"
+ " <SplashScreen Image='%ls\\splash-620x300.png'/>\r\n"
" </VisualElements>\r\n"
"</Application>";
@@ -331,7 +331,8 @@ bool CreateVisualElementsManifest(const base::FilePath& src_path,
// Fill the manifest with the desired values.
base::string16 manifest16(base::StringPrintf(
- manifest_template.c_str(), display_name.c_str(), elements_dir.c_str()));
+ manifest_template.c_str(), display_name.c_str(), elements_dir.c_str(),
+ elements_dir.c_str(), elements_dir.c_str()));
// Write the manifest to |src_path|.
const std::string manifest(base::UTF16ToUTF8(manifest16));

Powered by Google App Engine
This is Rietveld 408576698