Providing more information on why certain users can't be added to multi-profile

ash/session/session_state_delegate.h

Index: ash/session/session_state_delegate.h
diff --git a/ash/session/session_state_delegate.h b/ash/session/session_state_delegate.h
index 89c1ff8d4f951d2bd76630ebddfaf86f17107b8c..58cbf08ac8bf07074c68186a2e3c8562e4f9950f 100644
--- a/ash/session/session_state_delegate.h
+++ b/ash/session/session_state_delegate.h
@@ -65,6 +65,12 @@ class ASH_EXPORT SessionStateDelegate {
     SESSION_STATE_LOGIN_SECONDARY,
   };
 
+  enum PrimaryUserPolicy {
+    PRIMARY_USER_POLICY_ALLOW,
+    PRIMARY_USER_POLICY_FORBIDS,
+    PRIMARY_USER_POLICY_CERT_TAINTED,
+  };

[dzhioev (left Google), 2014/08/06 12:37:39]

Why do we need PrimaryUserPolicy enum, if we don't make distinctions in handling
of PRIMARY_USER_POLICY_FORBIDS and PRIMARY_USER_POLICY_CERT_TAINTED? Can we
change method "GetPrimaryUserPolicy" to something like "bool
IsMultiProfileAllowedByPolicy()"?

[Roman Sorokin (ftl), 2014/08/08 09:22:00]

Done.

+
   virtual ~SessionStateDelegate() {};
 
   // Returns the browser context for the user given by |index|.
@@ -134,6 +140,9 @@ class ASH_EXPORT SessionStateDelegate {
   // ordering as GetLoggedInUsers.
   virtual void CycleActiveUser(CycleUser cycle_user) = 0;
 
+  // Returns primary user policy.
+  virtual PrimaryUserPolicy GetPrimaryUserPolicy() const = 0;
+
   // Adds or removes sessions state observer.
   virtual void AddSessionStateObserver(SessionStateObserver* observer) = 0;
   virtual void RemoveSessionStateObserver(SessionStateObserver* observer) = 0;