Chromium Code Reviews| Index: chrome/browser/supervised_user/child_accounts/child_account_service.cc |
| diff --git a/chrome/browser/supervised_user/child_accounts/child_account_service.cc b/chrome/browser/supervised_user/child_accounts/child_account_service.cc |
| index db64a83d0b035cadc8b675c99550848074c2abd5..8fa14774db6eeeefb12011e340d7f7d9b4f1c1f5 100644 |
| --- a/chrome/browser/supervised_user/child_accounts/child_account_service.cc |
| +++ b/chrome/browser/supervised_user/child_accounts/child_account_service.cc |
| @@ -294,15 +294,13 @@ void ChildAccountService::SetIsChildAccount(bool is_child_account) { |
| void ChildAccountService::PropagateChildStatusToUser(bool is_child) { |
| #if defined(OS_CHROMEOS) |
| - // TODO(merkulova,treib): Figure out why this causes tests to fail. |
| -// user_manager::User* user = |
| -// chromeos::ProfileHelper::Get()->GetUserByProfile(profile_); |
| -// if (user) { |
| -// user_manager::UserManager::Get()->ChangeUserSupervisedStatus( |
| -// user, is_child); |
| -// } else { |
| -// LOG(WARNING) << |
| -// "User instance wasn't found while setting child account flag."; |
| -// } |
| + user_manager::User* user = |
|
Marc Treib
2014/12/08 13:27:54
This'll need a rebase on top of https://codereview
merkulova
2014/12/08 14:56:12
Definitely it won't be committed until that CL lan
Marc Treib
2014/12/08 15:00:29
Nah that's fine, just rebase when that one has lan
|
| + chromeos::ProfileHelper::Get()->GetUserByProfile(profile_); |
| + if (user) { |
| + user_manager::UserManager::Get()->ChangeUserChildStatus(user, is_child); |
| + } else { |
| + LOG(WARNING) << |
| + "User instance wasn't found while setting child account flag."; |
| + } |
| #endif |
| } |