arc: bluetooth: Implement set discoverable state

components/arc/bluetooth/arc_bluetooth_bridge.cc

 // Copyright 2016 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 "components/arc/bluetooth/arc_bluetooth_bridge.h"
 
 #include <bluetooth/bluetooth.h>
 #include <fcntl.h>
 #include <stddef.h>
 #include <sys/socket.h>
@@ skipping 600 matching lines @@
   if (!HasBluetoothInstance())
     return;
 
   mojo::Array<mojom::BluetoothPropertyPtr> properties =
       GetAdapterProperties(type);
 
   arc_bridge_service()->bluetooth()->instance()->OnAdapterProperties(
       mojom::BluetoothStatus::SUCCESS, std::move(properties));
 }
 
+void ArcBluetoothBridge::OnAdapterPropertySuccess(
+    mojo::Array<mojom::BluetoothPropertyPtr> properties) const {
+  arc_bridge_service()->bluetooth()->instance()->OnAdapterProperties(
+      mojom::BluetoothStatus::SUCCESS, std::move(properties));
+}
+
+void ArcBluetoothBridge::OnAdapterPropertyError(
+    mojom::BluetoothStatus status) const {
+  arc_bridge_service()->bluetooth()->instance()->OnAdapterProperties(
+      status, mojo::Array<mojom::BluetoothPropertyPtr>::New(0));
+}
+
 void ArcBluetoothBridge::SetAdapterProperty(
     mojom::BluetoothPropertyPtr property) {
   DCHECK(bluetooth_adapter_);
   if (!HasBluetoothInstance())
     return;
 
-  // TODO(smbarber): Implement SetAdapterProperty
-  arc_bridge_service()->bluetooth()->instance()->OnAdapterProperties(
-      mojom::BluetoothStatus::FAIL,
-      mojo::Array<mojom::BluetoothPropertyPtr>::New(0));
+  // TODO(puthik) Implement other case.

[Miao, 2016/08/30 22:05:38]

nit: Move this comment to the unsupported case.

+  if (property->is_discovery_timeout()) {
+    uint32_t discovery_timeout = property->get_discovery_timeout();
+    mojo::Array<mojom::BluetoothPropertyPtr> success_result;
+    success_result.push_back(std::move(property));
+    if (discovery_timeout > 0) {

[Miao, 2016/08/30 22:05:38]

nit: put a new line above.

+      // TODO(puthik) Add timer and turn this to off when timer end
+      bluetooth_adapter_->SetDiscoverable(
+          true,
+          base::Bind(&ArcBluetoothBridge::OnAdapterPropertySuccess,
+                     weak_factory_.GetWeakPtr(), base::Passed(&success_result)),
+          base::Bind(&ArcBluetoothBridge::OnAdapterPropertyError,
+                     weak_factory_.GetWeakPtr(), mojom::BluetoothStatus::FAIL));
+    } else {
+      OnAdapterPropertyError(mojom::BluetoothStatus::PARM_INVALID);
+    }
+  } else {
+    OnAdapterPropertyError(mojom::BluetoothStatus::UNSUPPORTED);
+  }
 }
 
 void ArcBluetoothBridge::GetRemoteDeviceProperty(
     mojom::BluetoothAddressPtr remote_addr,
     mojom::BluetoothPropertyType type) {
   DCHECK(bluetooth_adapter_);
   if (!HasBluetoothInstance())
     return;
 
   std::string addr_str = remote_addr->To<std::string>();
@@ skipping 1111 matching lines @@
   LOG(WARNING) << "Bluetooth instance is too old (version " << version
                << ") need version " << version_need;
   return false;
 }
 
 bool ArcBluetoothBridge::CalledOnValidThread() {
   return thread_checker_.CalledOnValidThread();
 }
 
 }  // namespace arc