Track whether a given user session has completed initialization, and use

components/user_manager/known_user.h

Index: components/user_manager/known_user.h
diff --git a/components/user_manager/known_user.h b/components/user_manager/known_user.h
index 245ed573f3872f388bda6525a0fbc0584b588c72..356300d5e84f17ff7f8a8750d8548ecce66e1b87 100644
--- a/components/user_manager/known_user.h
+++ b/components/user_manager/known_user.h
@@ -127,6 +127,17 @@ void USER_MANAGER_EXPORT UpdateUsingSAML(const AccountId& account_id,
 // returns false.
 bool USER_MANAGER_EXPORT IsUsingSAML(const AccountId& account_id);
 
+// Returns true if the user's session has already completed initialization
+// (set to false when session is created, and then is set to true once
+// the profile is intiaiized - this allows us to detect crashes/restarts during
+// initial session creation so we can recover gracefully).
+bool USER_MANAGER_EXPORT WasProfileEverInitialized(const AccountId& account_id);
+
+// Sets the flag that denotes whether the session associated with a user has
+// completed initialization at least once.
+void USER_MANAGER_EXPORT SetProfileEverInitialized(const AccountId& account_id,
+                                                   bool initialized);
+
 // Saves why the user has to go through re-auth flow.
 void USER_MANAGER_EXPORT UpdateReauthReason(const AccountId& account_id,
                                             const int reauth_reason);
@@ -138,8 +149,8 @@ bool USER_MANAGER_EXPORT FindReauthReason(const AccountId& account_id,
                                           int* out_value);
 
 // Removes all user preferences associated with |account_id|.
-// (This one used by user_manager only and thus not exported.)
-void RemovePrefs(const AccountId& account_id);
+// (exported so it can be used in tests).
+void USER_MANAGER_EXPORT RemovePrefs(const AccountId& account_id);

[Alexander Alekseev, 2017/03/09 02:04:04]

nit: Could you leave this private, and create a new (exported) proxy method
RemovePrefsForTesting(...), so that our presubmit hooks will work, preventing
RemovePrefsForTesting() this to be called from non-test code outside of
components/user_manager ?

[Andrew T Wilson (Slow), 2017/03/09 12:33:15]

Done.

 
 // Register known user prefs.
 void USER_MANAGER_EXPORT RegisterPrefs(PrefRegistrySimple* registry);