summaryrefslogtreecommitdiffstats
path: root/chrome/installer
diff options
context:
space:
mode:
authorevan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-04 21:15:34 +0000
committerevan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-08-04 21:15:34 +0000
commitdee7c0509d0e98d1210725aad2ce6e5153780592 (patch)
tree2408eb938674cde03bffb69350171980222d1584 /chrome/installer
parentf756abf74be7ce47d482f9760eacb608c9f0e121 (diff)
downloadchromium_src-dee7c0509d0e98d1210725aad2ce6e5153780592.zip
chromium_src-dee7c0509d0e98d1210725aad2ce6e5153780592.tar.gz
chromium_src-dee7c0509d0e98d1210725aad2ce6e5153780592.tar.bz2
Remove GetSwitchValue() from chrome/* where easy.
Review URL: http://codereview.chromium.org/3057033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54966 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/installer')
-rw-r--r--chrome/installer/setup/setup_main.cc23
-rw-r--r--chrome/installer/util/logging_installer.cc11
-rw-r--r--chrome/installer/util/logging_installer.h3
-rw-r--r--chrome/installer/util/master_preferences.cc8
4 files changed, 24 insertions, 21 deletions
diff --git a/chrome/installer/setup/setup_main.cc b/chrome/installer/setup/setup_main.cc
index 08e9ae5..b2e66a3 100644
--- a/chrome/installer/setup/setup_main.cc
+++ b/chrome/installer/setup/setup_main.cc
@@ -232,7 +232,7 @@ installer_util::InstallStatus InstallChrome(const CommandLine& cmd_line,
// If --install-archive is given, get the user specified value
if (cmd_line.HasSwitch(installer_util::switches::kInstallArchive)) {
- archive = cmd_line.GetSwitchValue(
+ archive = cmd_line.GetSwitchValueNative(
installer_util::switches::kInstallArchive);
}
LOG(INFO) << "Archive found to install Chrome " << archive;
@@ -288,7 +288,7 @@ installer_util::InstallStatus InstallChrome(const CommandLine& cmd_line,
// uncompressing and binary patching. Get the location for this file.
std::wstring archive_to_copy(temp_path.ToWStringHack());
file_util::AppendToPath(&archive_to_copy, installer::kChromeArchive);
- std::wstring prefs_source_path = cmd_line.GetSwitchValue(
+ std::wstring prefs_source_path = cmd_line.GetSwitchValueNative(
installer_util::switches::kInstallerData);
install_status = installer::InstallOrUpdateChrome(
cmd_line.program(), archive_to_copy, temp_path.ToWStringHack(),
@@ -355,7 +355,7 @@ installer_util::InstallStatus InstallChrome(const CommandLine& cmd_line,
LOG(INFO) << "Deleting temporary directory " << temp_path.value();
bool cleanup_success = file_util::Delete(temp_path, true);
if (cmd_line.HasSwitch(installer_util::switches::kInstallerData)) {
- std::wstring prefs_path = cmd_line.GetSwitchValue(
+ std::wstring prefs_path = cmd_line.GetSwitchValueNative(
installer_util::switches::kInstallerData);
cleanup_success = file_util::Delete(prefs_path, true) && cleanup_success;
}
@@ -369,7 +369,7 @@ installer_util::InstallStatus InstallChrome(const CommandLine& cmd_line,
if (!cleanup_success) {
ScheduleDirectoryForDeletion(temp_path.ToWStringHack().c_str());
if (cmd_line.HasSwitch(installer_util::switches::kInstallerData)) {
- std::wstring prefs_path = cmd_line.GetSwitchValue(
+ std::wstring prefs_path = cmd_line.GetSwitchValueNative(
installer_util::switches::kInstallerData);
ScheduleDirectoryForDeletion(prefs_path.c_str());
}
@@ -447,14 +447,14 @@ bool HandleNonInstallCmdLineOptions(const CommandLine& cmd_line,
if (!file_util::CreateNewTempDirectory(L"chrome_", &temp_path)) {
LOG(ERROR) << "Could not create temporary path.";
} else {
- std::wstring setup_patch = cmd_line.GetSwitchValue(
+ std::wstring setup_patch = cmd_line.GetSwitchValueNative(
installer_util::switches::kUpdateSetupExe);
LOG(INFO) << "Opening archive " << setup_patch;
std::wstring uncompressed_patch;
if (LzmaUtil::UnPackArchive(setup_patch, temp_path.ToWStringHack(),
&uncompressed_patch) == NO_ERROR) {
std::wstring old_setup_exe = cmd_line.program();
- std::wstring new_setup_exe = cmd_line.GetSwitchValue(
+ std::wstring new_setup_exe = cmd_line.GetSwitchValueNative(
installer_util::switches::kNewSetupExe);
if (!setup_util::ApplyDiffPatch(old_setup_exe, uncompressed_patch,
new_setup_exe))
@@ -474,7 +474,7 @@ bool HandleNonInstallCmdLineOptions(const CommandLine& cmd_line,
// Check if we need to show the EULA. If it is passed as a command line
// then the dialog is shown and regardless of the outcome setup exits here.
std::wstring inner_frame =
- cmd_line.GetSwitchValue(installer_util::switches::kShowEula);
+ cmd_line.GetSwitchValueNative(installer_util::switches::kShowEula);
exit_code = ShowEULADialog(inner_frame);
if (installer_util::EULA_REJECTED != exit_code)
GoogleUpdateSettings::SetEULAConsent(true);
@@ -486,12 +486,12 @@ bool HandleNonInstallCmdLineOptions(const CommandLine& cmd_line,
// browser for Start Menu->Internet shortcut. This option should only
// be used when setup.exe is launched with admin rights. We do not
// make any user specific changes in this option.
- std::wstring chrome_exe(cmd_line.GetSwitchValue(
+ std::wstring chrome_exe(cmd_line.GetSwitchValueNative(
installer_util::switches::kRegisterChromeBrowser));
std::wstring suffix;
if (cmd_line.HasSwitch(
installer_util::switches::kRegisterChromeBrowserSuffix)) {
- suffix = cmd_line.GetSwitchValue(
+ suffix = cmd_line.GetSwitchValueNative(
installer_util::switches::kRegisterChromeBrowserSuffix);
}
exit_code = ShellUtil::RegisterChromeBrowser(chrome_exe, suffix, false);
@@ -510,7 +510,7 @@ bool HandleNonInstallCmdLineOptions(const CommandLine& cmd_line,
std::wstring suffix;
if (cmd_line.HasSwitch(
installer_util::switches::kRegisterChromeBrowserSuffix)) {
- suffix = cmd_line.GetSwitchValue(
+ suffix = cmd_line.GetSwitchValueNative(
installer_util::switches::kRegisterChromeBrowserSuffix);
}
installer_util::InstallStatus tmp = installer_util::UNKNOWN_STATUS;
@@ -521,7 +521,8 @@ bool HandleNonInstallCmdLineOptions(const CommandLine& cmd_line,
} else if (cmd_line.HasSwitch(installer_util::switches::kInactiveUserToast)) {
// Launch the inactive user toast experiment.
std::wstring flavor =
- cmd_line.GetSwitchValue(installer_util::switches::kInactiveUserToast);
+ cmd_line.GetSwitchValueNative(
+ installer_util::switches::kInactiveUserToast);
int flavor_int;
base::StringToInt(flavor, &flavor_int);
dist->InactiveUserToastExperiment(flavor_int,
diff --git a/chrome/installer/util/logging_installer.cc b/chrome/installer/util/logging_installer.cc
index 3f11eb5..9dfda73 100644
--- a/chrome/installer/util/logging_installer.cc
+++ b/chrome/installer/util/logging_installer.cc
@@ -29,7 +29,7 @@ void InitInstallerLogging(const CommandLine& command_line) {
return;
}
- logging::InitLogging(GetLogFilePath(command_line).c_str(),
+ logging::InitLogging(GetLogFilePath(command_line).value().c_str(),
logging::LOG_ONLY_TO_FILE,
logging::LOCK_LOG_FILE,
logging::DELETE_OLD_LOG_FILE);
@@ -50,10 +50,10 @@ void EndInstallerLogging() {
installer_logging_ = false;
}
-std::wstring GetLogFilePath(const CommandLine& command_line) {
+FilePath GetLogFilePath(const CommandLine& command_line) {
if (command_line.HasSwitch(
WideToASCII(installer_util::switches::kLogFile))) {
- return command_line.GetSwitchValue(
+ return command_line.GetSwitchValuePath(
WideToASCII(installer_util::switches::kLogFile));
}
@@ -65,12 +65,11 @@ std::wstring GetLogFilePath(const CommandLine& command_line) {
}
FilePath log_path;
-
if (PathService::Get(base::DIR_TEMP, &log_path)) {
log_path = log_path.Append(log_filename);
- return log_path.ToWStringHack();
+ return log_path;
} else {
- return log_filename;
+ return FilePath(log_filename);
}
}
diff --git a/chrome/installer/util/logging_installer.h b/chrome/installer/util/logging_installer.h
index c2b0a96..50882dc 100644
--- a/chrome/installer/util/logging_installer.h
+++ b/chrome/installer/util/logging_installer.h
@@ -11,6 +11,7 @@
#include "base/logging.h"
class CommandLine;
+class FilePath;
namespace installer {
@@ -21,7 +22,7 @@ void InitInstallerLogging(const CommandLine& command_line);
void EndInstallerLogging();
// Returns the full path of the log file.
-std::wstring GetLogFilePath(const CommandLine& command_line);
+FilePath GetLogFilePath(const CommandLine& command_line);
} // namespace installer
diff --git a/chrome/installer/util/master_preferences.cc b/chrome/installer/util/master_preferences.cc
index 053dc45..63967d0 100644
--- a/chrome/installer/util/master_preferences.cc
+++ b/chrome/installer/util/master_preferences.cc
@@ -5,6 +5,7 @@
#include "chrome/installer/util/master_preferences.h"
#include "base/file_util.h"
+#include "base/string_util.h"
#include "base/logging.h"
#include "base/path_service.h"
#include "chrome/common/json_value_serializer.h"
@@ -105,9 +106,10 @@ bool GetDistroIntegerPreference(const DictionaryValue* prefs,
DictionaryValue* GetInstallPreferences(const CommandLine& cmd_line) {
DictionaryValue* prefs = NULL;
#if defined(OS_WIN)
- if (cmd_line.HasSwitch(installer_util::switches::kInstallerData)) {
- FilePath prefs_path(
- cmd_line.GetSwitchValue(installer_util::switches::kInstallerData));
+ std::string installer_data =
+ WideToASCII(installer_util::switches::kInstallerData);
+ if (cmd_line.HasSwitch(installer_data)) {
+ FilePath prefs_path = cmd_line.GetSwitchValuePath(installer_data);
prefs = installer_util::ParseDistributionPreferences(prefs_path);
}