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

Unified Diff: components/proximity_auth/device_to_device_authenticator.h

Issue 2561203002: Migrate weave-related classes from proximity_auth/ble to cryptauth/ble. (Closed)
Patch Set: Rebase. Created 4 years 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: components/proximity_auth/device_to_device_authenticator.h
diff --git a/components/proximity_auth/device_to_device_authenticator.h b/components/proximity_auth/device_to_device_authenticator.h
index 2b2ad64268806bcab5771a4d3a1c49f162fd1bcd..7172c7c4edf0f06b0684337e9ffeb99a3e72e7d9 100644
--- a/components/proximity_auth/device_to_device_authenticator.h
+++ b/components/proximity_auth/device_to_device_authenticator.h
@@ -8,8 +8,9 @@
#include "base/callback.h"
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
+#include "components/cryptauth/connection.h"
+#include "components/cryptauth/connection_observer.h"
#include "components/proximity_auth/authenticator.h"
-#include "components/proximity_auth/connection_observer.h"
namespace base {
class Timer;
@@ -21,8 +22,6 @@ class SecureMessageDelegate;
namespace proximity_auth {
-class Connection;
-
// Authenticator implementation using the "device to device" protocol, which is
// in turn built on top of the SecureMessage library.
// This protocol contains the following steps (local device is the initiator):
@@ -42,7 +41,7 @@ class Connection;
// This protocol requires exclusive use of the connection. No other message
// should be sent or received while authentication is in progress.
class DeviceToDeviceAuthenticator : public Authenticator,
- public ConnectionObserver {
+ public cryptauth::ConnectionObserver {
public:
// Creates the instance:
// |connection|: The connection to the remote device, which must be in a
@@ -50,7 +49,7 @@ class DeviceToDeviceAuthenticator : public Authenticator,
// |account_id|: The canonical account id of the user who is the owner of both
// the local and remote devices.
// |secure_message_delegate|: Handles the SecureMessage crypto operations.
- DeviceToDeviceAuthenticator(Connection* connection,
+ DeviceToDeviceAuthenticator(cryptauth::Connection* connection,
const std::string& account_id,
std::unique_ptr<cryptauth::SecureMessageDelegate>
secure_message_delegate);
@@ -108,19 +107,20 @@ class DeviceToDeviceAuthenticator : public Authenticator,
void Succeed();
// ConnectionObserver:
- void OnConnectionStatusChanged(Connection* connection,
- Connection::Status old_status,
- Connection::Status new_status) override;
- void OnMessageReceived(const Connection& connection,
- const WireMessage& message) override;
- void OnSendCompleted(const Connection& connection,
- const WireMessage& message,
+ void OnConnectionStatusChanged(
+ cryptauth::Connection* connection,
+ cryptauth::Connection::Status old_status,
+ cryptauth::Connection::Status new_status) override;
+ void OnMessageReceived(const cryptauth::Connection& connection,
+ const cryptauth::WireMessage& message) override;
+ void OnSendCompleted(const cryptauth::Connection& connection,
+ const cryptauth::WireMessage& message,
bool success) override;
// The connection to the remote device. It is expected to be in the CONNECTED
// state at all times during authentication.
// Not owned, and must outlive this instance.
- Connection* const connection_;
+ cryptauth::Connection* const connection_;
// The account id of the user who owns the local and remote devices. This is
// normally an email address, and should be canonicalized.

Powered by Google App Engine
This is Rietveld 408576698