Add easyUnlockPrivate.getConnection API.

chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc

Index: chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc
diff --git a/chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc b/chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc
index 6021e28fa026c304e7e85f335715e7399b3b2bd8..3a5175e868c463d8340b8f8882864a264360a529 100644
--- a/chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc
+++ b/chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.cc
@@ -710,5 +710,48 @@ bool EasyUnlockPrivateGetUserImageFunction::RunSync() {
   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