Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2085)

Unified Diff: chrome/browser/chromeos/login/user_manager_impl.h

Issue 63173012: SupervisedUsers : Fix transaction cleanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/user_manager_impl.h
diff --git a/chrome/browser/chromeos/login/user_manager_impl.h b/chrome/browser/chromeos/login/user_manager_impl.h
index df38699fa77e073918b12660dc05292170173815..94a910a68b4b9132955eefb66282189b52767693 100644
--- a/chrome/browser/chromeos/login/user_manager_impl.h
+++ b/chrome/browser/chromeos/login/user_manager_impl.h
@@ -185,6 +185,10 @@ class UserManagerImpl
// list. Returns |NULL| otherwise.
const User* FindUserInList(const std::string& user_id) const;
+ // Returns |true| if user with the given id is found in the persistent list.
+ // Returns |false| otherwise. Does not trigger user loading.
+ const bool UserExistsInList(const std::string& user_id) const;
+
// Same as FindUserInList but returns non-const pointer to User object.
User* FindUserInListAndModify(const std::string& user_id);
@@ -316,6 +320,17 @@ class UserManagerImpl
Profile* GetProfileByUser(const User* user) const;
+ // Implementation for RemoveUser method. This is an asynchronous part of the
+ // method, that verifies that owner will not get deleted, and calls
+ // |RemoveNonOwnerUserInternal|.
+ void RemoveUserInternal(const std::string& user_email,
+ RemoveUserDelegate* delegate);
+
+ // Implementation for RemoveUser method. It is synchronous. It is called from
+ // RemoveUserInternal after owner check.
+ void RemoveNonOwnerUserInternal(const std::string& user_email,
+ RemoveUserDelegate* delegate);
+
// MultiProfileUserControllerDelegate implementation:
virtual void OnUserNotAllowed() OVERRIDE;
« no previous file with comments | « chrome/browser/chromeos/login/supervised_user_manager_impl.cc ('k') | chrome/browser/chromeos/login/user_manager_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698