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

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

Issue 2801353002: [CrOS Tether] Fill out the Initializer class. Tether will now initialize fully once the flag is ena… (Closed)
Patch Set: Added missing dep. 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/components/tether/tether_connector.cc ('k') | chromeos/components/tether/tether_host_fetcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/components/tether/tether_host_fetcher.h
diff --git a/chromeos/components/tether/tether_host_fetcher.h b/chromeos/components/tether/tether_host_fetcher.h
index d190758c9821b9096a4fab593a941e6e6f5c3ff0..a5ce6ae764f2d58596ae5f002fed22892f1a254e 100644
--- a/chromeos/components/tether/tether_host_fetcher.h
+++ b/chromeos/components/tether/tether_host_fetcher.h
@@ -28,10 +28,7 @@ namespace tether {
// synced via CryptAuth.
class TetherHostFetcher {
public:
- TetherHostFetcher(const std::string& user_id,
- const std::string& user_private_key,
- cryptauth::CryptAuthService* cryptauth_service,
- cryptauth::CryptAuthDeviceManager* device_manager);
+ explicit TetherHostFetcher(cryptauth::CryptAuthService* cryptauth_service);
virtual ~TetherHostFetcher();
// Fetches all tether hosts.
@@ -47,7 +44,8 @@ class TetherHostFetcher {
protected:
struct TetherHostFetchRequest {
- TetherHostFetchRequest(const TetherHostListCallback& list_callback);
+ explicit TetherHostFetchRequest(
+ const TetherHostListCallback& list_callback);
TetherHostFetchRequest(const std::string& device_id,
const TetherHostCallback& single_callback);
TetherHostFetchRequest(const TetherHostFetchRequest& other);
@@ -63,6 +61,11 @@ class TetherHostFetcher {
TetherHostCallback single_callback;
};
+ TetherHostFetcher(const std::string& user_id,
+ const std::string& user_private_key,
+ cryptauth::CryptAuthService* cryptauth_service,
+ cryptauth::CryptAuthDeviceManager* device_manager);
+
void OnRemoteDevicesLoaded(const cryptauth::RemoteDeviceList& remote_devices);
std::vector<TetherHostFetchRequest> requests_;
@@ -70,11 +73,7 @@ class TetherHostFetcher {
private:
void StartLoadingDevicesIfNeeded();
- const std::string user_id_;
- const std::string user_private_key_;
-
cryptauth::CryptAuthService* cryptauth_service_;
- cryptauth::CryptAuthDeviceManager* device_manager_;
std::unique_ptr<cryptauth::RemoteDeviceLoader> remote_device_loader_;
« no previous file with comments | « chromeos/components/tether/tether_connector.cc ('k') | chromeos/components/tether/tether_host_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698