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

Unified Diff: chrome/installer/util/master_preferences_unittest.cc

Issue 4988001: Revert 66088 - Changing the installer switches from wchar_t[] to char[].... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 1 month 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
« no previous file with comments | « chrome/installer/util/master_preferences.cc ('k') | chrome/installer/util/shell_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/util/master_preferences_unittest.cc
===================================================================
--- chrome/installer/util/master_preferences_unittest.cc (revision 66088)
+++ chrome/installer/util/master_preferences_unittest.cc (working copy)
@@ -303,14 +303,15 @@
}
TEST_F(MasterPreferencesTest, TestDefaultInstallConfig) {
- std::wstringstream chrome_cmd, cf_cmd, ceee_cmd;
- chrome_cmd << "setup.exe";
- cf_cmd << "setup.exe --" << installer_util::switches::kChromeFrame;
- ceee_cmd << "setup.exe --" << installer_util::switches::kCeee;
+ const std::wstring kChromeInstall(L"setup.exe");
+ const std::wstring kCfInstall(StringPrintf(L"setup.exe --%ls",
+ installer_util::switches::kChromeFrame));
+ const std::wstring kCeeeInstall(StringPrintf(L"setup.exe --%ls",
+ installer_util::switches::kCeee));
- CommandLine chrome_install(CommandLine::FromString(chrome_cmd.str()));
- CommandLine cf_install(CommandLine::FromString(cf_cmd.str()));
- CommandLine ceee_install(CommandLine::FromString(ceee_cmd.str()));
+ CommandLine chrome_install(CommandLine::FromString(kChromeInstall));
+ CommandLine cf_install(CommandLine::FromString(kCfInstall));
+ CommandLine ceee_install(CommandLine::FromString(kCeeeInstall));
installer_util::MasterPreferences pref_chrome(chrome_install);
installer_util::MasterPreferences pref_cf(cf_install);
@@ -333,27 +334,33 @@
}
TEST_F(MasterPreferencesTest, TestMultiInstallConfig) {
- using installer_util::switches::kMultiInstall;
- using installer_util::switches::kChrome;
- using installer_util::switches::kChromeFrame;
- using installer_util::switches::kCeee;
+ const std::wstring kChromeInstall(StringPrintf(L"setup.exe --%ls --%ls",
+ installer_util::switches::kMultiInstall,
+ installer_util::switches::kChrome));
+ const std::wstring kCfInstall(StringPrintf(L"setup.exe --%ls --%ls",
+ installer_util::switches::kMultiInstall,
+ installer_util::switches::kChromeFrame));
+ const std::wstring kCeeeInstall(StringPrintf(L"setup.exe --%ls --%ls",
+ installer_util::switches::kMultiInstall,
+ installer_util::switches::kCeee));
+ const std::wstring kChromeCfInstall(
+ StringPrintf(L"setup.exe --%ls --%ls --%ls",
+ installer_util::switches::kMultiInstall,
+ installer_util::switches::kChrome,
+ installer_util::switches::kChromeFrame));
+ const std::wstring kChromeCeeeCfInstall(
+ StringPrintf(L"setup.exe --%ls --%ls --%ls --%ls",
+ installer_util::switches::kMultiInstall,
+ installer_util::switches::kChrome,
+ installer_util::switches::kChromeFrame,
+ installer_util::switches::kCeee));
- std::wstringstream chrome_cmd, cf_cmd, ceee_cmd, chrome_cf_cmd,
- chrome_ceee_cf_cmd;
- chrome_cmd << "setup.exe --" << kMultiInstall << " --" << kChrome;
- cf_cmd << "setup.exe --" << kMultiInstall << " --" << kChromeFrame;
- ceee_cmd << "setup.exe --" << kMultiInstall << " --" << kCeee;
- chrome_cf_cmd << "setup.exe --" << kMultiInstall << " --" << kChrome <<
- " --" << kChromeFrame;
- chrome_ceee_cf_cmd << "setup.exe --" << kMultiInstall << " --" << kChrome <<
- " --" << kChromeFrame << " --" << kCeee;
-
- CommandLine chrome_install(CommandLine::FromString(chrome_cmd.str()));
- CommandLine cf_install(CommandLine::FromString(cf_cmd.str()));
- CommandLine ceee_install(CommandLine::FromString(ceee_cmd.str()));
- CommandLine chrome_cf_install(CommandLine::FromString(chrome_cf_cmd.str()));
+ CommandLine chrome_install(CommandLine::FromString(kChromeInstall));
+ CommandLine cf_install(CommandLine::FromString(kCfInstall));
+ CommandLine ceee_install(CommandLine::FromString(kCeeeInstall));
+ CommandLine chrome_cf_install(CommandLine::FromString(kChromeCfInstall));
CommandLine chrome_cf_ceee_install(
- CommandLine::FromString(chrome_ceee_cf_cmd.str()));
+ CommandLine::FromString(kChromeCeeeCfInstall));
installer_util::MasterPreferences pref_chrome(chrome_install);
installer_util::MasterPreferences pref_cf(cf_install);
« no previous file with comments | « chrome/installer/util/master_preferences.cc ('k') | chrome/installer/util/shell_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698