HostScanner: Add scanned hosts to NetworkStateHandler and NotificationPresenter.

chromeos/components/tether/host_scanner.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chromeos/components/tether/host_scanner.h"
 
 #include "base/bind.h"
 #include "chromeos/components/tether/tether_host_fetcher.h"
+#include "chromeos/network/network_state.h"
 #include "components/cryptauth/remote_device_loader.h"
 
 namespace chromeos {
 
 namespace tether {
 
 HostScanner::HostScanner(
     TetherHostFetcher* tether_host_fetcher,
     BleConnectionManager* connection_manager,
-    HostScanDevicePrioritizer* host_scan_device_prioritizer)
+    HostScanDevicePrioritizer* host_scan_device_prioritizer,
+    NetworkStateHandler* network_state_handler,
+    NotificationPresenter* notification_presenter)
     : tether_host_fetcher_(tether_host_fetcher),
       connection_manager_(connection_manager),
       host_scan_device_prioritizer_(host_scan_device_prioritizer),
+      network_state_handler_(network_state_handler),
+      notification_presenter_(notification_presenter),
       is_fetching_hosts_(false),
       weak_ptr_factory_(this) {}
 
 HostScanner::~HostScanner() {}
 
 void HostScanner::StartScan() {
   if (host_scanner_operation_) {
     // If a scan is already active, do not continue.
     return;
   }
@@ skipping 21 matching lines @@
   host_scanner_operation_->AddObserver(this);
   host_scanner_operation_->Initialize();
 }
 
 void HostScanner::OnTetherAvailabilityResponse(
     std::vector<HostScannerOperation::ScannedDeviceInfo>&
         scanned_device_list_so_far,
     bool is_final_scan_result) {
   most_recent_scan_results_ = scanned_device_list_so_far;
 
-  // TODO(hansberry): Hook up to networking code.
+  if (!scanned_device_list_so_far.empty()) {
+    if (scanned_device_list_so_far.size() == 1) {

[Kyle Horimoto, 2017/04/07 01:08:07]

nit: Do the UI stuff after the networks have been added instead of before.

[Ryan Hansberry, 2017/04/07 01:25:04]

Done.

+      notification_presenter_->NotifyPotentialHotspotNearby(
+          scanned_device_list_so_far.at(0).remote_device);
+    } else {
+      notification_presenter_->NotifyMultiplePotentialHotspotsNearby();
+    }
+
+    // TODO (hansberry): Clear out old scanned hosts from NetworkStateHandler.

[Kyle Horimoto, 2017/04/07 01:08:08]

You'll also need a TODO for adding the device information (battery, reception,
etc).

[Ryan Hansberry, 2017/04/07 01:25:04]

Done.

+    for (auto scanned_device_info : scanned_device_list_so_far) {
+      cryptauth::RemoteDevice remote_device = scanned_device_info.remote_device;
+      network_state_handler_->AddTetherNetworkState(remote_device.GetDeviceId(),
+                                                    remote_device.name);
+    }
+  }
 
   if (is_final_scan_result) {
     // If the final scan result has been received, the operation is finished.
     // Delete it.
     host_scanner_operation_->RemoveObserver(this);
     host_scanner_operation_.reset();
   }
 }
 
 }  // namespace tether
 
 }  // namespace chromeos