Made possible to lock and unlock with several logged in users.

Made possible to lock and unlock with several logged in users.

Every signed in user can lock and unlock screen.
User entering password becomes active after unlock.
Changed only backend part. UI changes and tests will follow.

BUG=239228
TEST=manually
TBR=darin

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203712

[Nikita (slow), 2013-06-03 09:04:15]

lgtm with nits

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
File chrome/browser/chromeos/login/screen_locker.cc (right):

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:279: LOG(ERROR) << "Invalid user
trying to unlock.";
CHECK as well?

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:294: CHECK(users_.size() == 1);
LOG(ERROR) before check?

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:462: 
nit: This file now has 2 empty lines at the end?

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
File chrome/browser/chromeos/login/screen_locker.h (right):

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.h:41: explicit ScreenLocker(const
UserList& users);
nit: Please add commend about |users| parameter.

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
File chrome/browser/chromeos/login/webui_screen_locker.cc (right):

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/webui_screen_locker.cc:75: // User list consisting
of a single logged-in user.
nit: Remove comment.

[dzhioev (left Google), 2013-06-03 09:39:57]

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
File chrome/browser/chromeos/login/screen_locker.cc (right):

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:279: LOG(ERROR) << "Invalid user
trying to unlock.";
On 2013/06/03 09:04:15, Nikita Kostylev wrote:
> CHECK as well?

Done. Replaced with LOG_ASSERT.

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:294: CHECK(users_.size() == 1);
On 2013/06/03 09:04:15, Nikita Kostylev wrote:
> LOG(ERROR) before check?

Done. Replaced with LOG_ASSERT.

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/screen_locker.cc:462: 
On 2013/06/03 09:04:15, Nikita Kostylev wrote:
> nit: This file now has 2 empty lines at the end?
No, only one.

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
File chrome/browser/chromeos/login/webui_screen_locker.cc (right):

https://codereview.chromium.org/16278003/diff/1/chrome/browser/chromeos/login...
chrome/browser/chromeos/login/webui_screen_locker.cc:75: // User list consisting
of a single logged-in user.
On 2013/06/03 09:04:15, Nikita Kostylev wrote:
> nit: Remove comment.

Done.

[commit-bot: I haz the power, 2013-06-03 09:40:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dzhioev@chromium.org/16278003/1002

[commit-bot: I haz the power, 2013-06-03 09:47:58]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-03 10:31:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dzhioev@chromium.org/16278003/1002

[commit-bot: I haz the power, 2013-06-03 14:11:50]

Change committed as 203712