cros: Move IsUserSupervised and IsUserChild off SystemTrayDelegate

ash/session/session_controller.h

Index: ash/session/session_controller.h
diff --git a/ash/session/session_controller.h b/ash/session/session_controller.h
index 5500b410eaa2aa120d25f978bbf2d7b27413b0f6..fa594e1e8afe14234a5ba3f47f194a003c0356d0 100644
--- a/ash/session/session_controller.h
+++ b/ash/session/session_controller.h
@@ -85,6 +85,13 @@ class ASH_EXPORT SessionController
   // nullptr if no user session is found for the index.
   const mojom::UserSession* GetUserSession(UserIndex index) const;
 
+  // Returns true if the current user is supervised: has legacy supervised
+  // account or kid account.
+  bool IsUserSupervised() const;
+
+  // Returns true if the current user is a child account.
+  bool IsUserChild() const;

[James Cook, 2017/04/19 20:31:57]

I just moved the method here because it was simple. However, I wonder if it
would be better to provide a GetActiveUserType() method. That might help reduce
usage of ash::LoginStatus. OTOH, the method would have to return something when
there was no active user, which would mean adding UserType::UNKNOWN = -1 (since
there are histograms on UserType) and updating the mojo serializers. WDYT?

[xiyuan, 2017/04/19 20:57:59]

SGTM. It would be a good starer bug. :p

[James Cook, 2017/04/19 22:16:37]

Filed crbug.com/713379

+
   // Locks the screen. The locking happens asynchronously.
   void LockScreen();