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

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

Issue 11267023: Implementing --app-launcher install/uninstall flow. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nits; added localized string for App Launcher uninstall shortcut. Created 8 years, 2 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/util/master_preferences.cc
diff --git a/chrome/installer/util/master_preferences.cc b/chrome/installer/util/master_preferences.cc
index 07459c61af9ba51aa49b3f3d6128e1713ec15902..9678a40fd8c1e730110d59dd21fcca0b7e713b82 100644
--- a/chrome/installer/util/master_preferences.cc
+++ b/chrome/installer/util/master_preferences.cc
@@ -93,8 +93,11 @@ MasterPreferences::MasterPreferences(const CommandLine& cmd_line)
}
MasterPreferences::MasterPreferences(const FilePath& prefs_path)
- : distribution_(NULL), preferences_read_from_file_(false),
- chrome_(true), chrome_app_host_(false), chrome_frame_(false),
+ : distribution_(NULL),
+ preferences_read_from_file_(false),
+ chrome_(true),
+ chrome_app_host_(false),
+ chrome_frame_(false),
multi_install_(false) {
std::string json_data;
// Failure to read the file is ignored as |json_data| will be the empty string
@@ -141,6 +144,8 @@ void MasterPreferences::InitializeFromCommandLine(const CommandLine& cmd_line) {
installer::master_preferences::kAutoLaunchChrome },
{ installer::switches::kChromeAppHost,
installer::master_preferences::kChromeAppHost },
+ { installer::switches::kChromeAppLauncher,
+ installer::master_preferences::kChromeAppLauncher },
{ installer::switches::kChrome,
installer::master_preferences::kChrome },
{ installer::switches::kChromeFrame,
@@ -221,6 +226,8 @@ void MasterPreferences::InitializeFromString(const std::string& json_data) {
}
void MasterPreferences::InitializeProductFlags() {
+ bool pref_chrome_app_host = false;
+ bool pref_chrome_app_launcher = false;
// Make sure we start out with the correct defaults.
multi_install_ = false;
chrome_frame_ = false;
@@ -229,7 +236,11 @@ void MasterPreferences::InitializeProductFlags() {
GetBool(installer::master_preferences::kMultiInstall, &multi_install_);
GetBool(installer::master_preferences::kChromeFrame, &chrome_frame_);
- GetBool(installer::master_preferences::kChromeAppHost, &chrome_app_host_);
+
+ GetBool(installer::master_preferences::kChromeAppHost, &pref_chrome_app_host);
+ GetBool(installer::master_preferences::kChromeAppLauncher,
+ &pref_chrome_app_launcher);
+ chrome_app_host_ = pref_chrome_app_host || pref_chrome_app_launcher;
// When multi-install is specified, the checks are pretty simple (in theory):
// In order to be installed/uninstalled, each product must have its switch
« no previous file with comments | « chrome/installer/util/installation_validator.cc ('k') | chrome/installer/util/master_preferences_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698