| Index: chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
|
| diff --git a/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc b/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
|
| index 054f8fe6f8d340cdd5405231faa2bd27e486ccd0..44b31160ebe7b38af82ab7c562c5c47f8ac4fe6a 100644
|
| --- a/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
|
| +++ b/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
|
| @@ -40,7 +40,6 @@
|
| #include "chromeos/cryptohome/async_method_caller.h"
|
| #include "chromeos/cryptohome/cryptohome_parameters.h"
|
| #include "chromeos/dbus/dbus_thread_manager.h"
|
| -#include "chromeos/dbus/session_manager_client.h"
|
| #include "chromeos/settings/cros_settings_names.h"
|
| #include "components/ownership/owner_key_util.h"
|
| #include "components/prefs/pref_registry_simple.h"
|
| @@ -52,7 +51,6 @@
|
| #include "content/public/browser/browser_thread.h"
|
| #include "extensions/common/extension_urls.h"
|
| #include "extensions/common/manifest_handlers/kiosk_mode_info.h"
|
| -#include "third_party/cros_system_api/switches/chrome_switches.h"
|
|
|
| namespace chromeos {
|
|
|
| @@ -149,14 +147,6 @@
|
| minor_version, bugfix_version));
|
| }
|
|
|
| -// Converts a flag constant to actual command line switch value.
|
| -std::string GetSwitchString(const std::string& flag_name) {
|
| - base::CommandLine cmd_line(base::CommandLine::NO_PROGRAM);
|
| - cmd_line.AppendSwitch(flag_name);
|
| - DCHECK_EQ(2U, cmd_line.argv().size());
|
| - return cmd_line.argv()[1];
|
| -}
|
| -
|
| } // namespace
|
|
|
| // static
|
| @@ -251,68 +241,8 @@
|
| const std::string& app_id) {
|
| LOG_IF(FATAL, app_session_) << "Kiosk session is already initialized.";
|
|
|
| - base::CommandLine session_flags(base::CommandLine::NO_PROGRAM);
|
| - if (GetSwitchesForSessionRestore(app_id, &session_flags)) {
|
| - base::CommandLine::StringVector flags;
|
| - // argv[0] is the program name |base::CommandLine::NO_PROGRAM|.
|
| - flags.assign(session_flags.argv().begin() + 1, session_flags.argv().end());
|
| -
|
| - // Update user flags, but do not restart Chrome - the purpose of the flags
|
| - // set here is to be able to properly restore session if the session is
|
| - // restarted - e.g. due to crash. For example, this will ensure restarted
|
| - // app session restores auto-launched state.
|
| - DBusThreadManager::Get()->GetSessionManagerClient()->SetFlagsForUser(
|
| - cryptohome::Identification(
|
| - user_manager::UserManager::Get()->GetActiveUser()->GetAccountId()),
|
| - flags);
|
| - }
|
| -
|
| app_session_.reset(new AppSession);
|
| app_session_->Init(profile, app_id);
|
| -}
|
| -
|
| -bool KioskAppManager::GetSwitchesForSessionRestore(
|
| - const std::string& app_id,
|
| - base::CommandLine* switches) {
|
| - bool auto_launched = app_id == currently_auto_launched_with_zero_delay_app_;
|
| - const base::CommandLine* current_command_line =
|
| - base::CommandLine::ForCurrentProcess();
|
| - bool has_auto_launched_flag =
|
| - current_command_line->HasSwitch(switches::kAppAutoLaunched);
|
| - if (auto_launched == has_auto_launched_flag)
|
| - return false;
|
| -
|
| - // Collect current policy defined switches, so they can be passed on to the
|
| - // session manager as well - otherwise they would get lost on restart.
|
| - // This ignores 'flag-switches-begin' - 'flag-switches-end' flags, but those
|
| - // should not be present for kiosk sessions.
|
| - bool in_policy_switches_block = false;
|
| - const std::string policy_switches_begin =
|
| - GetSwitchString(switches::kPolicySwitchesBegin);
|
| - const std::string policy_switches_end =
|
| - GetSwitchString(switches::kPolicySwitchesEnd);
|
| -
|
| - for (const auto& it : current_command_line->argv()) {
|
| - if (it == policy_switches_begin) {
|
| - DCHECK(!in_policy_switches_block);
|
| - in_policy_switches_block = true;
|
| - }
|
| -
|
| - if (in_policy_switches_block)
|
| - switches->AppendSwitch(it);
|
| -
|
| - if (it == policy_switches_end) {
|
| - DCHECK(in_policy_switches_block);
|
| - in_policy_switches_block = false;
|
| - }
|
| - }
|
| -
|
| - DCHECK(!in_policy_switches_block);
|
| -
|
| - if (auto_launched)
|
| - switches->AppendSwitch(switches::kAppAutoLaunched);
|
| -
|
| - return true;
|
| }
|
|
|
| void KioskAppManager::AddAppForTest(
|
|
|