bluetooth: Implement RSSI and Tx Power on macOS and Android

device/bluetooth/bluetooth_device.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 "device/bluetooth/bluetooth_device.h"
 
 #include <iterator>
 #include <memory>
 #include <string>
 
@@ skipping 293 matching lines @@
 
 const std::vector<uint8_t>* BluetoothDevice::GetServiceDataForUUID(
     const BluetoothUUID& uuid) const {
   auto it = service_data_.find(uuid);
   if (it != service_data_.end()) {
     return &it->second;
   }
   return nullptr;
 }
 
+base::Optional<int8_t> BluetoothDevice::GetInquiryRSSI() const {
+  return inquiry_rssi_;
+}
+
+base::Optional<int8_t> BluetoothDevice::GetInquiryTxPower() const {
+  return inquiry_tx_power_;
+}
+
 void BluetoothDevice::CreateGattConnection(
     const GattConnectionCallback& callback,
     const ConnectErrorCallback& error_callback) {
   create_gatt_connection_success_callbacks_.push_back(callback);
   create_gatt_connection_error_callbacks_.push_back(error_callback);
 
   if (IsGattConnected())
     return DidConnectGatt();
 
   CreateGattConnectionImpl();
@@ skipping 50 matching lines @@
 
       canonicalized[i] = base::ToUpperASCII(canonicalized[i]);
     }
   }
 
   return canonicalized;
 }
 
 std::string BluetoothDevice::GetIdentifier() const { return GetAddress(); }
 
-void BluetoothDevice::UpdateAdvertisementData(UUIDList advertised_uuids,
-                                              ServiceDataMap service_data) {
+void BluetoothDevice::UpdateAdvertisementData(int8_t rssi,
+                                              UUIDList advertised_uuids,
+                                              ServiceDataMap service_data,
+                                              const int8_t* tx_power) {
   UpdateTimestamp();
+
+  inquiry_rssi_ = rssi;
+
   device_uuids_.ReplaceAdvertisedUUIDs(std::move(advertised_uuids));
   service_data_ = std::move(service_data);
+
+  if (tx_power != nullptr) {
+    inquiry_tx_power_ = *tx_power;
+  } else {
+    inquiry_tx_power_ = base::nullopt;
+  }
 }
 
 void BluetoothDevice::ClearAdvertisementData() {
+  inquiry_rssi_ = base::nullopt;
   device_uuids_.ClearAdvertisedUUIDs();
   service_data_.clear();
+  inquiry_tx_power_ = base::nullopt;
   GetAdapter()->NotifyDeviceChanged(this);
 }
 
 void BluetoothDevice::DidConnectGatt() {
   for (const auto& callback : create_gatt_connection_success_callbacks_) {
     callback.Run(
         base::WrapUnique(new BluetoothGattConnection(adapter_, GetAddress())));
   }
   create_gatt_connection_success_callbacks_.clear();
   create_gatt_connection_error_callbacks_.clear();
@@ skipping 46 matching lines @@
 void BluetoothDevice::Pair(PairingDelegate* pairing_delegate,
                            const base::Closure& callback,
                            const ConnectErrorCallback& error_callback) {
   NOTREACHED();
 }
 
 void BluetoothDevice::UpdateTimestamp() {
   last_update_time_ = base::Time::NowFromSystemTime();
 }
 
+// static
+int8_t BluetoothDevice::ClampPower(int power) {
+  if (power < INT8_MIN) {

[Jeffrey Yasskin, 2016/08/24 04:32:02]

Do you want to let -128 through or restrict this to -127..127?

[ortuno, 2016/08/24 21:29:09]

I thought it would be weird if someone added -128 in the adv packet and saw -127
on device.txPower so I just allowed the whole int8 range.

+    return INT8_MIN;
+  }
+  if (power > INT8_MAX) {
+    return INT8_MAX;
+  }
+  return static_cast<int8_t>(power);
+}
+
 }  // namespace device