Implemented DeviceStatusCollector::GetDeviceSessionStatus()

chrome/browser/chromeos/login/users/mock_user_manager.cc

Index: chrome/browser/chromeos/login/users/mock_user_manager.cc
diff --git a/chrome/browser/chromeos/login/users/mock_user_manager.cc b/chrome/browser/chromeos/login/users/mock_user_manager.cc
index 19a887193df90cc58555e160d02a93b034e2daf6..d09c1fc7f61fb65bbe3855d3d4e6a68915386077 100644
--- a/chrome/browser/chromeos/login/users/mock_user_manager.cc
+++ b/chrome/browser/chromeos/login/users/mock_user_manager.cc
@@ -108,6 +108,15 @@ user_manager::User* MockUserManager::CreatePublicAccountUser(
   return user_list_.back();
 }
 
+user_manager::User* MockUserManager::CreateKioskAppUser(
+    const std::string& email) {
+  ClearUserList();
+  user_manager::User* user = user_manager::User::CreateKioskAppUser(email);

[bartfab (slow), 2015/02/11 17:28:42]

Nit: s/user_manager::User* user/user_manager::User* const user/

[Andrew T Wilson (Slow), 2015/02/12 15:15:38]

Removed

+  user_list_.push_back(user);

[bartfab (slow), 2015/02/11 17:28:43]

Nit: Why not push_back() the User object as it is created, and then refer to it
via user_list_.back() throughout? This avoids the short stretch of code where
|user| is owned via a raw pointer.

[Andrew T Wilson (Slow), 2015/02/12 15:15:38]

Done.

+  ProfileHelper::Get()->SetProfileToUserMappingForTesting(user);
+  return user_list_.back();
+}
+
 void MockUserManager::AddUser(const std::string& email) {
   user_manager::User* user = user_manager::User::CreateRegularUser(email);
   user_list_.push_back(user);