Expose device RSSI and Tx power via the chrome.bluetooth API.

chrome/browser/extensions/api/bluetooth/bluetooth_api_utils.cc

 // Copyright (c) 2012 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/bluetooth/bluetooth_api_utils.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/extensions/api/bluetooth.h"
 #include "device/bluetooth/bluetooth_adapter.h"
@@ skipping 94 matching lines @@
     out->vendor_id.reset(new int(device.GetVendorID()));
     out->product_id.reset(new int(device.GetProductID()));
     out->device_id.reset(new int(device.GetDeviceID()));
   }
 
   ConvertDeviceTypeToApi(device.GetDeviceType(), &(out->type));
 
   out->paired.reset(new bool(device.IsPaired()));
   out->connected.reset(new bool(device.IsConnected()));
 
+  if (*out->connected) {
+    int rssi = device.GetRSSI();

[keybuk, 2014/04/27 09:08:44]

RSSI can also be provided when not connected - e.g. RSSI of received Inquiry
results and AD

We should provide those too, and they can go in the same property

[Ilya Sherman, 2014/04/29 00:51:17]

I extended the IDL documentation to specify this, and moved the GetRSSI() call
out of the if stmt.  However, I don't plan to implement fetching the RSSI from
received Inquiry results as part of this CL, because I don't know how to read
that data on Mac.  I instead added a NOTIMPLEMENTED() for that case, as a
reminder to come back to this at a later time.  Does that sound reasonable?

+    if (rssi != BluetoothDevice::kUnknownPower)
+      out->rssi.reset(new int(rssi));
+    int current_transmit_power = device.GetCurrentHostTransmitPower();
+    if (current_transmit_power != BluetoothDevice::kUnknownPower)
+      out->current_host_transmit_power.reset(new int(current_transmit_power));
+    int maximum_transmit_power = device.GetMaximumHostTransmitPower();
+    if (maximum_transmit_power != BluetoothDevice::kUnknownPower)
+      out->maximum_host_transmit_power.reset(new int(maximum_transmit_power));

[keybuk, 2014/04/27 09:08:44]

These on the other hand only make sense when connected

(Inquiry / Advertising TX Power can go in an Adapter property)

[Ilya Sherman, 2014/04/29 00:51:17]

Ack.

+  }
+
   std::vector<std::string>* string_uuids = new std::vector<std::string>();
   const device::BluetoothDevice::UUIDList& uuids = device.GetUUIDs();
   for (device::BluetoothDevice::UUIDList::const_iterator iter = uuids.begin();
        iter != uuids.end(); ++iter)
     string_uuids->push_back(iter->canonical_value());
   out->uuids.reset(string_uuids);
 }
 
 void PopulateAdapterState(const device::BluetoothAdapter& adapter,
                           AdapterState* out) {
   out->discovering = adapter.IsDiscovering();
   out->available = adapter.IsPresent();
   out->powered = adapter.IsPowered();
   out->name = adapter.GetName();
   out->address = adapter.GetAddress();
 }
 
 }  // namespace bluetooth
 }  // namespace api
 }  // namespace extensions