Add easyUnlockPrivate.getConnection API.

chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.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 "chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
@@ skipping 692 matching lines @@
       reinterpret_cast<const char*>(user_image->front()), user_image->size()));
 #else
   // TODO(tengs): Find a way to get the profile picture for non-ChromeOS
   // devices.
   results_ = easy_unlock_private::GetUserImage::Results::Create("");
   SetError("Not supported on non-ChromeOS platforms.");
 #endif
   return true;
 }
 
+EasyUnlockPrivateGetConnectionInfoFunction::
+    EasyUnlockPrivateGetConnectionInfoFunction() {
+}
+
+EasyUnlockPrivateGetConnectionInfoFunction::
+    ~EasyUnlockPrivateGetConnectionInfoFunction() {
+}
+
+bool EasyUnlockPrivateGetConnectionInfoFunction::DoWork(
+    scoped_refptr<device::BluetoothAdapter> adapter) {
+  scoped_ptr<easy_unlock_private::GetConnectionInfo::Params> params =
+      easy_unlock_private::GetConnectionInfo::Params::Create(*args_);
+  EXTENSION_FUNCTION_VALIDATE(params);
+
+  device::BluetoothDevice* device = adapter->GetDevice(params->device_address);
+
+  std::string error;
+  if (!device)
+    error = "Invalid Bluetooth device.";
+  else if (!device->IsConnected())
+    error = "Bluetooth device not connected.";
+
+  if (!error.empty()) {
+    SetError(error);
+    SendResponse(false);
+    return true;

[Yoyo Zhou, 2015/01/08 20:23:05]

Aside: It looks like BluetoothExtensionFunction just ignores these return
values, and they're not documented either =(. Why do you return the values you
have chosen here?

[Tim Song, 2015/01/09 00:31:46]

I followed the styles of the other subclasses that return true if the API
function call completes after the DoWork() call. This is similar to
AsyncExtensionFunction::Run().

Maybe the return value was relevant in the past and is redundant now due to
refactoring?

+  }
+
+  device->GetConnectionInfo(base::Bind(
+      &EasyUnlockPrivateGetConnectionInfoFunction::OnConnectionInfo, this));
+  return false;
+}
+
+void EasyUnlockPrivateGetConnectionInfoFunction::OnConnectionInfo(
+    const device::BluetoothDevice::ConnectionInfo& connection_info) {
+  scoped_ptr<base::ListValue> results(new base::ListValue());
+  results->AppendInteger(connection_info.rssi);
+  results->AppendInteger(connection_info.transmit_power);
+  results->AppendInteger(connection_info.max_transmit_power);
+  SetResultList(results.Pass());
+  SendResponse(true);
+}
+
 }  // namespace api
 }  // namespace extensions