ARC: Implement the startScan API to start scanning of WiFI APs.

components/arc/net/arc_net_host_impl.cc

Index: components/arc/net/arc_net_host_impl.cc
diff --git a/components/arc/net/arc_net_host_impl.cc b/components/arc/net/arc_net_host_impl.cc
index bca78b13c03447ef42e2e64027f8a2d93671a298..f7a4aa5073fad6091563aa85e004d60937d59b5d 100644
--- a/components/arc/net/arc_net_host_impl.cc
+++ b/components/arc/net/arc_net_host_impl.cc
@@ -26,6 +26,10 @@ const int kGetNetworksListLimit = 100;
 
 namespace arc {
 
+chromeos::NetworkStateHandler* GetStateHandler() {
+  return chromeos::NetworkHandler::Get()->network_state_handler();
+}
+
 ArcNetHostImpl::ArcNetHostImpl(ArcBridgeService* bridge_service)
     : ArcService(bridge_service), binding_(this) {
   arc_bridge_service()->AddObserver(this);
@@ -100,12 +104,15 @@ void ArcNetHostImpl::GetNetworks(bool configured_only,
 
 void ArcNetHostImpl::GetWifiEnabledState(
     const GetWifiEnabledStateCallback& callback) {
-  chromeos::NetworkStateHandler* handler =
-      chromeos::NetworkHandler::Get()->network_state_handler();
-  bool is_enabled =
-      handler->IsTechnologyEnabled(chromeos::NetworkTypePattern::WiFi());
+  bool is_enabled = GetStateHandler()->IsTechnologyEnabled(
+      chromeos::NetworkTypePattern::WiFi());
 
   callback.Run(is_enabled);
 }
 
+void ArcNetHostImpl::StartScan(const StartScanCallback& callback) {
+  GetStateHandler()->RequestScan();
+  callback.Run(true);

[Luis Héctor Chávez, 2016/02/08 16:35:46]

Why add this if it is always going to be true?

If you want to have a callback when it is actually run, you can declare it like
so:

StartScan() => ();

[abhishekbh, 2016/02/12 02:17:33]

Removed.

+}
+
 }  // namespace arc