Adapt Chrome OS related part of ChildAccountService.

chrome/browser/supervised_user/child_accounts/child_account_service.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/supervised_user/child_accounts/child_account_service.h"
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
@@ skipping 276 matching lines @@
     profile_->GetPrefs()->SetString(prefs::kSupervisedUserId,
                                     kChildAccountSUID);
   } else {
     profile_->GetPrefs()->ClearPref(prefs::kSupervisedUserId);
   }
   PropagateChildStatusToUser(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 =
+      chromeos::ProfileHelper::Get()->GetUserByProfile(profile_);
+  if (!user) {
+    LOG(WARNING) <<
+        "User instance wasn't found while setting child account flag.";
+  } else if (user->GetType() == user_manager::USER_TYPE_REGULAR_SUPERVISED ||

[Marc Treib, 2014/12/08 16:59:34]

Hm, I don't think we should check the user type here. After all, if we're trying
to set "isChild" to true for a different type of user, then something is wrong,
and we'd like to hit the check in user.cc. Or am I missing something?

[Marc Treib, 2014/12/10 12:42:36]

This one's still not addressed.

+             user->GetType() == user_manager::USER_TYPE_REGULAR) {
+    user_manager::UserManager::Get()->ChangeUserSupervisedStatus(
+        user, is_child);
+  }
 #endif
 }