Commit ccf0a3a1 authored by antrim's avatar antrim Committed by Commit bot

Additional logging for crash mentioned in crbug.com/422918

R=nkostylev@chromium.org

Review URL: https://codereview.chromium.org/666443002

Cr-Commit-Position: refs/heads/master@{#301407}
parent 859369ba
......@@ -135,7 +135,7 @@ UserContext ChromeLoginPerformer::TransformSupervisedKey(
void ChromeLoginPerformer::SetupSupervisedUserFlow(const std::string& user_id) {
SupervisedUserLoginFlow* new_flow = new SupervisedUserLoginFlow(user_id);
new_flow->set_host(ChromeUserManager::Get()->GetUserFlow(user_id)->host());
new_flow->SetHost(ChromeUserManager::Get()->GetUserFlow(user_id)->host());
ChromeUserManager::Get()->SetUserFlow(user_id, new_flow);
}
......
......@@ -446,8 +446,11 @@ void ExistingUserController::Login(const UserContext& user_context,
void ExistingUserController::PerformLogin(
const UserContext& user_context,
LoginPerformer::AuthorizationMode auth_mode) {
ChromeUserManager::Get()->GetUserFlow(user_context.GetUserID())->set_host(
host_);
// TODO(antrim): remove this output once crash reason is found.
LOG(ERROR) << "Setting flow from PerformLogin";
ChromeUserManager::Get()
->GetUserFlow(user_context.GetUserID())
->SetHost(host_);
BootTimesLoader::Get()->RecordLoginAttempted();
......
......@@ -34,7 +34,9 @@ SupervisedUserCreationFlow::SupervisedUserCreationFlow(
session_started_(false),
manager_profile_(NULL) {}
SupervisedUserCreationFlow::~SupervisedUserCreationFlow() {}
SupervisedUserCreationFlow::~SupervisedUserCreationFlow() {
LOG(ERROR) << "Destroyed " << this;
}
bool SupervisedUserCreationFlow::CanLockScreen() {
return false;
......@@ -95,6 +97,8 @@ bool SupervisedUserCreationFlow::HandlePasswordChangeDetected() {
void SupervisedUserCreationFlow::LaunchExtraSteps(
Profile* profile) {
// TODO(antrim): remove this output once crash is found.
LOG(ERROR) << "LaunchExtraSteps for " << this << " host is " << host();
logged_in_ = true;
manager_profile_ = profile;
ProfileHelper::Get()->ProfileStartup(profile, true);
......
......@@ -22,6 +22,12 @@ UserFlow::UserFlow() : host_(NULL) {}
UserFlow::~UserFlow() {}
void UserFlow::SetHost(LoginDisplayHost* host) {
// TODO(antrim): remove this output once crash reason is found.
LOG(ERROR) << "Flow " << this << " got host " << host;
host_ = host;
}
DefaultUserFlow::~DefaultUserFlow() {}
void DefaultUserFlow::AppendAdditionalCommandLineSwitches() {
......
......@@ -37,10 +37,7 @@ class UserFlow {
virtual void HandleOAuthTokenStatusChange(
user_manager::User::OAuthTokenStatus status) = 0;
virtual void LaunchExtraSteps(Profile* profile) = 0;
void set_host(LoginDisplayHost* host) {
host_ = host;
}
void SetHost(LoginDisplayHost* host);
LoginDisplayHost* host() {
return host_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment