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

Unified Diff: chrome/browser/chromeos/login/users/mock_user_manager.h

Issue 417623002: user_manager component: Add UserManagerBase class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 months 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/users/mock_user_manager.h
diff --git a/chrome/browser/chromeos/login/users/mock_user_manager.h b/chrome/browser/chromeos/login/users/mock_user_manager.h
index 26aa3e6014c1ce9cdf4583ce06afe3ef2ffe752c..fb72cabe5f1b2027c0d2d20802233a509a1b1abc 100644
--- a/chrome/browser/chromeos/login/users/mock_user_manager.h
+++ b/chrome/browser/chromeos/login/users/mock_user_manager.h
@@ -78,11 +78,11 @@ class MockUserManager : public UserManager {
MOCK_CONST_METHOD0(AreSupervisedUsersAllowed, bool(void));
// You can't mock these functions easily because nobody can create
- // User objects but the UserManagerImpl and us.
+ // User objects but the ChromeUserManager and us.
virtual const user_manager::UserList& GetUsers() const OVERRIDE;
virtual const user_manager::User* GetLoggedInUser() const OVERRIDE;
virtual user_manager::UserList GetUnlockUsers() const OVERRIDE;
- virtual const std::string& GetOwnerEmail() OVERRIDE;
+ virtual const std::string& GetOwnerEmail() const OVERRIDE;
virtual user_manager::User* GetLoggedInUser() OVERRIDE;
Dmitry Polukhin 2014/07/28 09:32:39 Why it is no const?
Nikita (slow) 2014/07/29 09:27:28 Replied in another comment.
virtual const user_manager::User* GetActiveUser() const OVERRIDE;
virtual user_manager::User* GetActiveUser() OVERRIDE;
Dmitry Polukhin 2014/07/28 09:32:39 Ditto.
Nikita (slow) 2014/07/29 09:27:27 Replied in another comment.

Powered by Google App Engine
This is Rietveld 408576698