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

Unified Diff: chromeos/components/tether/ble_connection_manager.h

Issue 2800883003: [CrOS Tether] Update SecureChannel and BleConnectionManager to use CryptAuthService instead of a Del (Closed)
Patch Set: Added BleConnectionManager refactor as well. 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 | « no previous file | chromeos/components/tether/ble_connection_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/components/tether/ble_connection_manager.h
diff --git a/chromeos/components/tether/ble_connection_manager.h b/chromeos/components/tether/ble_connection_manager.h
index 96a01b88cdbfcfb9d3d2f7787573cee5ae61d806..3558d27479c38d376259a925e1a3ae8c66918793 100644
--- a/chromeos/components/tether/ble_connection_manager.h
+++ b/chromeos/components/tether/ble_connection_manager.h
@@ -22,6 +22,7 @@
namespace cryptauth {
class BluetoothThrottler;
+class CryptAuthService;
} // namespace cryptauth
namespace chromeos {
@@ -65,14 +66,8 @@ class BleConnectionManager : public BleScanner::Observer {
const std::string& payload) = 0;
};
- class Delegate {
- public:
- virtual std::unique_ptr<cryptauth::SecureChannel::Delegate>
- CreateSecureChannelDelegate() = 0;
- };
-
BleConnectionManager(
- std::unique_ptr<Delegate> delegate,
+ cryptauth::CryptAuthService* cryptauth_service,
scoped_refptr<device::BluetoothAdapter> adapter,
const LocalDeviceDataProvider* local_device_data_provider,
const cryptauth::RemoteBeaconSeedFetcher* remote_beacon_seed_fetcher,
@@ -180,7 +175,7 @@ class BleConnectionManager : public BleScanner::Observer {
};
BleConnectionManager(
- std::unique_ptr<Delegate> delegate,
+ cryptauth::CryptAuthService* cryptauth_service,
scoped_refptr<device::BluetoothAdapter> adapter,
std::unique_ptr<BleScanner> ble_scanner,
std::unique_ptr<BleAdvertiser> ble_advertiser,
@@ -206,7 +201,7 @@ class BleConnectionManager : public BleScanner::Observer {
const cryptauth::SecureChannel::Status& old_status,
const cryptauth::SecureChannel::Status& new_status);
- std::unique_ptr<Delegate> delegate_;
+ cryptauth::CryptAuthService* cryptauth_service_;
scoped_refptr<device::BluetoothAdapter> adapter_;
std::unique_ptr<BleScanner> ble_scanner_;
std::unique_ptr<BleAdvertiser> ble_advertiser_;
« no previous file with comments | « no previous file | chromeos/components/tether/ble_connection_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698