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

Unified Diff: chromeos/dbus/auth_policy_client.h

Issue 2794493002: Add AuthPolicyLoginHelper (Closed)
Patch Set: Update after review Created 3 years, 8 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
« no previous file with comments | « chromeos/BUILD.gn ('k') | chromeos/dbus/auth_policy_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/dbus/auth_policy_client.h
diff --git a/chromeos/dbus/auth_policy_client.h b/chromeos/dbus/auth_policy_client.h
index 99c9b811c6ccfbede43344faed4eb363cbe5bb29..4eb76e277a6071905db854c2d11153bbc165689b 100644
--- a/chromeos/dbus/auth_policy_client.h
+++ b/chromeos/dbus/auth_policy_client.h
@@ -24,11 +24,11 @@ class CHROMEOS_EXPORT AuthPolicyClient : public DBusClient {
public:
// |user_id| is a unique id for the users. Using objectGUID from Active
// Directory server.
- using AuthCallback = base::Callback<void(
+ using AuthCallback = base::OnceCallback<void(
authpolicy::ErrorType error,
const authpolicy::ActiveDirectoryAccountData& account_data)>;
- using JoinCallback = base::Callback<void(authpolicy::ErrorType error)>;
- using RefreshPolicyCallback = base::Callback<void(bool success)>;
+ using JoinCallback = base::OnceCallback<void(authpolicy::ErrorType error)>;
+ using RefreshPolicyCallback = base::OnceCallback<void(bool success)>;
~AuthPolicyClient() override;
@@ -46,7 +46,7 @@ class CHROMEOS_EXPORT AuthPolicyClient : public DBusClient {
virtual void JoinAdDomain(const std::string& machine_name,
const std::string& user_principal_name,
int password_fd,
- const JoinCallback& callback) = 0;
+ JoinCallback callback) = 0;
// Calls AuthenticateUser. It runs "kinit <user_principal_name> .. " which
// does kerberos authentication against Active Directory server.
@@ -54,17 +54,17 @@ class CHROMEOS_EXPORT AuthPolicyClient : public DBusClient {
// |callback| is called after getting (or failing to get) D-BUS response.
virtual void AuthenticateUser(const std::string& user_principal_name,
int password_fd,
- const AuthCallback& callback) = 0;
+ AuthCallback callback) = 0;
// Calls RefreshDevicePolicy - handle policy for the device.
// Fetch GPO files from Active directory server, parse it, encode it into
// protobuf and send to SessionManager. Callback is called after that.
- virtual void RefreshDevicePolicy(const RefreshPolicyCallback& callback) = 0;
+ virtual void RefreshDevicePolicy(RefreshPolicyCallback callback) = 0;
// Calls RefreshUserPolicy - handle policy for the user specified by
// |account_id|. Similar to RefreshDevicePolicy.
virtual void RefreshUserPolicy(const AccountId& account_id,
- const RefreshPolicyCallback& callback) = 0;
+ RefreshPolicyCallback callback) = 0;
protected:
// Create() should be used instead.
« no previous file with comments | « chromeos/BUILD.gn ('k') | chromeos/dbus/auth_policy_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698