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

Unified Diff: chromeos/components/tether/host_scanner.cc

Issue 2844973002: [CrOS Tether] Create TetherHostResponseRecorder, which records ConnectTetheringResponses and Tether… (Closed)
Patch Set: Changed "Connectable" wording to "Connected". 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/host_scanner.h ('k') | chromeos/components/tether/host_scanner_operation.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/components/tether/host_scanner.cc
diff --git a/chromeos/components/tether/host_scanner.cc b/chromeos/components/tether/host_scanner.cc
index 66312c3acb4edbd10c6f596d39f1897cf6edd154..e9e48d7edf55725713bb08fda93818d63357942a 100644
--- a/chromeos/components/tether/host_scanner.cc
+++ b/chromeos/components/tether/host_scanner.cc
@@ -36,12 +36,14 @@ HostScanner::HostScanner(
TetherHostFetcher* tether_host_fetcher,
BleConnectionManager* connection_manager,
HostScanDevicePrioritizer* host_scan_device_prioritizer,
+ TetherHostResponseRecorder* tether_host_response_recorder,
NetworkStateHandler* network_state_handler,
NotificationPresenter* notification_presenter,
DeviceIdTetherNetworkGuidMap* device_id_tether_network_guid_map)
: tether_host_fetcher_(tether_host_fetcher),
connection_manager_(connection_manager),
host_scan_device_prioritizer_(host_scan_device_prioritizer),
+ tether_host_response_recorder_(tether_host_response_recorder),
network_state_handler_(network_state_handler),
notification_presenter_(notification_presenter),
device_id_tether_network_guid_map_(device_id_tether_network_guid_map),
@@ -75,7 +77,8 @@ void HostScanner::OnTetherHostsFetched(
is_fetching_hosts_ = false;
host_scanner_operation_ = HostScannerOperation::Factory::NewInstance(
- tether_hosts, connection_manager_, host_scan_device_prioritizer_);
+ tether_hosts, connection_manager_, host_scan_device_prioritizer_,
+ tether_host_response_recorder_);
host_scanner_operation_->AddObserver(this);
host_scanner_operation_->Initialize();
}
@@ -115,6 +118,7 @@ void HostScanner::OnTetherAvailabilityResponse(
status.connection_strength())
: 100;
+ // TODO(khorimoto): Pass a HasConnectedToHost parameter to this function.
network_state_handler_->AddTetherNetworkState(
device_id_tether_network_guid_map_->GetTetherNetworkGuidForDeviceId(
remote_device.GetDeviceId()),
« no previous file with comments | « chromeos/components/tether/host_scanner.h ('k') | chromeos/components/tether/host_scanner_operation.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698