diff options
author | thakis <thakis@chromium.org> | 2015-02-05 19:42:14 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-02-06 03:44:25 +0000 |
commit | c0107ee59bdb5b6337f7bbdb4e8210c0bfeb547b (patch) | |
tree | 60e81dabaf818666ecb5940227b6b184e827b0ab /chrome/installer | |
parent | 8a0f4e28e2030d0a6ad09b68311efc428cc48ce6 (diff) | |
download | chromium_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.cc | 15 |
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)); |