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

Side by Side Diff: chrome/browser/chromeos/login/lock/screen_locker.cc

Issue 521463005: Make SetState be called later when screen is unlocking. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/login/lock/screen_locker.h" 5 #include "chrome/browser/chromeos/login/lock/screen_locker.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "ash/ash_switches.h" 10 #include "ash/ash_switches.h"
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 LOG(ERROR) << "Start time is not set at authentication success"; 198 LOG(ERROR) << "Start time is not set at authentication success";
199 } else { 199 } else {
200 base::TimeDelta delta = base::Time::Now() - authentication_start_time_; 200 base::TimeDelta delta = base::Time::Now() - authentication_start_time_;
201 VLOG(1) << "Authentication success: " << delta.InSecondsF() << " second(s)"; 201 VLOG(1) << "Authentication success: " << delta.InSecondsF() << " second(s)";
202 UMA_HISTOGRAM_TIMES("ScreenLocker.AuthenticationSuccessTime", delta); 202 UMA_HISTOGRAM_TIMES("ScreenLocker.AuthenticationSuccessTime", delta);
203 } 203 }
204 204
205 const user_manager::User* user = 205 const user_manager::User* user =
206 user_manager::UserManager::Get()->FindUser(user_context.GetUserID()); 206 user_manager::UserManager::Get()->FindUser(user_context.GetUserID());
207 if (user) { 207 if (user) {
208 if (user->is_active()) { 208 if (!user->is_active()) {
209 DCHECK(saved_ime_state_); 209 saved_ime_state_ = NULL;
210 input_method::InputMethodManager::Get()->SetState(saved_ime_state_);
211 } else {
212 user_manager::UserManager::Get()->SwitchActiveUser( 210 user_manager::UserManager::Get()->SwitchActiveUser(
213 user_context.GetUserID()); 211 user_context.GetUserID());
214 } 212 }
215 } else { 213 } else {
216 NOTREACHED() << "Logged in user not found."; 214 NOTREACHED() << "Logged in user not found.";
217 } 215 }
218 saved_ime_state_ = NULL;
219 216
220 authentication_capture_.reset(new AuthenticationParametersCapture()); 217 authentication_capture_.reset(new AuthenticationParametersCapture());
221 authentication_capture_->user_context = user_context; 218 authentication_capture_->user_context = user_context;
222 219
223 // Add guard for case when something get broken in call chain to unlock 220 // Add guard for case when something get broken in call chain to unlock
224 // for sure. 221 // for sure.
225 base::MessageLoop::current()->PostDelayedTask( 222 base::MessageLoop::current()->PostDelayedTask(
226 FROM_HERE, 223 FROM_HERE,
227 base::Bind(&ScreenLocker::UnlockOnLoginSuccess, 224 base::Bind(&ScreenLocker::UnlockOnLoginSuccess,
228 weak_factory_.GetWeakPtr()), 225 weak_factory_.GetWeakPtr()),
(...skipping 227 matching lines...) Expand 10 before | Expand all | Expand 10 after
456 bool state = false; 453 bool state = false;
457 VLOG(1) << "Emitting SCREEN_LOCK_STATE_CHANGED with state=" << state; 454 VLOG(1) << "Emitting SCREEN_LOCK_STATE_CHANGED with state=" << state;
458 content::NotificationService::current()->Notify( 455 content::NotificationService::current()->Notify(
459 chrome::NOTIFICATION_SCREEN_LOCK_STATE_CHANGED, 456 chrome::NOTIFICATION_SCREEN_LOCK_STATE_CHANGED,
460 content::Source<ScreenLocker>(this), 457 content::Source<ScreenLocker>(this),
461 content::Details<bool>(&state)); 458 content::Details<bool>(&state));
462 459
463 VLOG(1) << "Calling session manager's HandleLockScreenDismissed D-Bus method"; 460 VLOG(1) << "Calling session manager's HandleLockScreenDismissed D-Bus method";
464 DBusThreadManager::Get()->GetSessionManagerClient()-> 461 DBusThreadManager::Get()->GetSessionManagerClient()->
465 NotifyLockScreenDismissed(); 462 NotifyLockScreenDismissed();
463
464 if (saved_ime_state_) {
465 input_method::InputMethodManager::Get()->SetState(saved_ime_state_);
466 }
466 } 467 }
467 468
468 void ScreenLocker::SetAuthenticator(Authenticator* authenticator) { 469 void ScreenLocker::SetAuthenticator(Authenticator* authenticator) {
469 authenticator_ = authenticator; 470 authenticator_ = authenticator;
470 } 471 }
471 472
472 void ScreenLocker::ScreenLockReady() { 473 void ScreenLocker::ScreenLockReady() {
473 locked_ = true; 474 locked_ = true;
474 base::TimeDelta delta = base::Time::Now() - start_time_; 475 base::TimeDelta delta = base::Time::Now() - start_time_;
475 VLOG(1) << "ScreenLocker " << this << " is ready after " 476 VLOG(1) << "ScreenLocker " << this << " is ready after "
(...skipping 28 matching lines...) Expand all
504 for (user_manager::UserList::const_iterator it = users_.begin(); 505 for (user_manager::UserList::const_iterator it = users_.begin();
505 it != users_.end(); 506 it != users_.end();
506 ++it) { 507 ++it) {
507 if ((*it)->email() == username) 508 if ((*it)->email() == username)
508 return true; 509 return true;
509 } 510 }
510 return false; 511 return false;
511 } 512 }
512 513
513 } // namespace chromeos 514 } // namespace chromeos
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698