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

Unified Diff: chrome/browser/chromeos/login/managed/supervised_user_login_flow.cc

Issue 221813006: Various supervised user password fixes - 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: upload once again Created 6 years, 9 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/browser/chromeos/login/managed/supervised_user_login_flow.cc
diff --git a/chrome/browser/chromeos/login/managed/supervised_user_login_flow.cc b/chrome/browser/chromeos/login/managed/supervised_user_login_flow.cc
index d28681f09e7d3af4a102f3f161c7c2983a62eaa1..e48c0114df3b4cdd93e0eec374c71df09f478c6c 100644
--- a/chrome/browser/chromeos/login/managed/supervised_user_login_flow.cc
+++ b/chrome/browser/chromeos/login/managed/supervised_user_login_flow.cc
@@ -111,7 +111,7 @@ void SupervisedUserLoginFlow::OnPasswordChangeDataLoaded(
int revision = 0;
int schema = 0;
bool success = password_data->GetStringWithoutPathExpansion(
- kPasswordSignatureKey, &base64_signature);
+ kPasswordSignature, &base64_signature);
success &= password_data->GetIntegerWithoutPathExpansion(kPasswordRevision,
&revision);
success &=
@@ -181,7 +181,7 @@ void SupervisedUserLoginFlow::OnNewKeyAdded(
SupervisedUserAuthentication* auth =
UserManager::Get()->GetSupervisedUserManager()->GetAuthentication();
auth->StorePasswordData(user_id(), *password_data.get());
- auth->MarkKeyIncomplete(user_id());
+ auth->MarkKeyIncomplete(user_id(), true);
Nikita (slow) 2014/04/02 14:18:20 nit: true /* incomplete */
Denis Kuznetsov (DE-MUC) 2014/04/02 14:48:51 Done.
authenticator_->RemoveKey(
context_,
kLegacyCryptohomeManagedUserKeyLabel,
@@ -230,7 +230,7 @@ void SupervisedUserLoginFlow::OnPasswordUpdated(
bool was_incomplete = auth->HasIncompleteKey(user_id());
auth->StorePasswordData(user_id(), *password_data.get());
if (was_incomplete)
- auth->MarkKeyIncomplete(user_id());
+ auth->MarkKeyIncomplete(user_id(), true);
Nikita (slow) 2014/04/02 14:18:20 nit: true /* incomplete */
Denis Kuznetsov (DE-MUC) 2014/04/02 14:48:51 Done.
UMA_HISTOGRAM_ENUMERATION(
"ManagedUsers.ChromeOS.PasswordChange",

Powered by Google App Engine
This is Rietveld 408576698