summaryrefslogtreecommitdiffstats
path: root/chrome/installer
diff options
context:
space:
mode:
authorthakis <thakis@chromium.org>2015-02-05 19:42:14 -0800
committerCommit bot <commit-bot@chromium.org>2015-02-06 03:44:25 +0000
commitc0107ee59bdb5b6337f7bbdb4e8210c0bfeb547b (patch)
tree60e81dabaf818666ecb5940227b6b184e827b0ab /chrome/installer
parent8a0f4e28e2030d0a6ad09b68311efc428cc48ce6 (diff)
downloadchromium_src-c0107ee59bdb5b6337f7bbdb4e8210c0bfeb547b.zip
chromium_src-c0107ee59bdb5b6337f7bbdb4e8210c0bfeb547b.tar.gz
chromium_src-c0107ee59bdb5b6337f7bbdb4e8210c0bfeb547b.tar.bz2
Revert "Enable positional parameters for base::vsnprintf and base::vswprintf on Windows."
This reverts r127788. Positional parameters aren't part of the C standard (they are part of POSIX though), we don't really use them in the codebase, and they don't work in chrome/android with libc++. Rather than have a test for them, make them also not work on Windows (like they originally didn't) -- we shouldn't use positional printf parameters anyways. Change the 5 uses to not use positional parameters. BUG=118064,117028,427718 Review URL: https://codereview.chromium.org/902643002 Cr-Commit-Position: refs/heads/master@{#314963}
Diffstat (limited to 'chrome/installer')
-rw-r--r--chrome/installer/setup/install.cc15
1 files changed, 8 insertions, 7 deletions
diff --git a/chrome/installer/setup/install.cc b/chrome/installer/setup/install.cc
index d94824d..68ecef4 100644
--- a/chrome/installer/setup/install.cc
+++ b/chrome/installer/setup/install.cc
@@ -285,20 +285,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>";
@@ -314,7 +314,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));