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

Side by Side Diff: chrome/browser/chromeos/login/auth/chrome_login_performer.h

Issue 1183893005: ChromeOS: user whitelist check should happen before entering password. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after review. Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_
6 #define CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_ 6 #define CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 16 matching lines...) Expand all
27 27
28 namespace chromeos { 28 namespace chromeos {
29 29
30 // This class implements chrome-specific elements of Login Performer. 30 // This class implements chrome-specific elements of Login Performer.
31 31
32 class ChromeLoginPerformer : public LoginPerformer { 32 class ChromeLoginPerformer : public LoginPerformer {
33 public: 33 public:
34 explicit ChromeLoginPerformer(Delegate* delegate); 34 explicit ChromeLoginPerformer(Delegate* delegate);
35 ~ChromeLoginPerformer() override; 35 ~ChromeLoginPerformer() override;
36 36
37 bool IsUserWhitelisted(const std::string& user_id,
38 bool* wildcard_match) override;
39
37 protected: 40 protected:
38 bool RunTrustedCheck(const base::Closure& callback) override; 41 bool RunTrustedCheck(const base::Closure& callback) override;
39 void DidRunTrustedCheck(const base::Closure& callback); 42 void DidRunTrustedCheck(const base::Closure& callback);
40 bool IsUserWhitelisted(const std::string& user_id,
41 bool* wildcard_match) override;
42 43
43 void RunOnlineWhitelistCheck(const std::string& user_id, 44 void RunOnlineWhitelistCheck(const std::string& user_id,
44 bool wildcard_match, 45 bool wildcard_match,
45 const std::string& refresh_token, 46 const std::string& refresh_token,
46 const base::Closure& success_callback, 47 const base::Closure& success_callback,
47 const base::Closure& failure_callback) override; 48 const base::Closure& failure_callback) override;
48 bool AreSupervisedUsersAllowed() override; 49 bool AreSupervisedUsersAllowed() override;
49 50
50 bool UseExtendedAuthenticatorForSupervisedUser( 51 bool UseExtendedAuthenticatorForSupervisedUser(
51 const UserContext& user_context) override; 52 const UserContext& user_context) override;
(...skipping 18 matching lines...) Expand all
70 // Used to verify logins that matched wildcard on the login whitelist. 71 // Used to verify logins that matched wildcard on the login whitelist.
71 scoped_ptr<policy::WildcardLoginChecker> wildcard_login_checker_; 72 scoped_ptr<policy::WildcardLoginChecker> wildcard_login_checker_;
72 base::WeakPtrFactory<ChromeLoginPerformer> weak_factory_; 73 base::WeakPtrFactory<ChromeLoginPerformer> weak_factory_;
73 74
74 DISALLOW_COPY_AND_ASSIGN(ChromeLoginPerformer); 75 DISALLOW_COPY_AND_ASSIGN(ChromeLoginPerformer);
75 }; 76 };
76 77
77 } // namespace chromeos 78 } // namespace chromeos
78 79
79 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_ 80 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_AUTH_CHROME_LOGIN_PERFORMER_H_
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/login/app_launch_signin_screen.cc ('k') | chrome/browser/chromeos/login/existing_user_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698