Hook up RSSI and host transmit power Bluetooth device properties for ChromeOS.

chromeos/dbus/bluetooth_device_client.h

 // Copyright 2013 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.
 
 #ifndef CHROMEOS_DBUS_BLUETOOTH_DEVICE_CLIENT_H_
 #define CHROMEOS_DBUS_BLUETOOTH_DEVICE_CLIENT_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 58 matching lines @@
     dbus::Property<dbus::ObjectPath> adapter;
 
     // Indicates whether the device is likely to only support pre-2.1
     // PIN Code pairing rather than 2.1 Secure Simple Pairing, this can
     // give false positives. Read-only.
     dbus::Property<bool> legacy_pairing;
 
     // Remote Device ID information in Linux kernel modalias format. Read-only.
     dbus::Property<std::string> modalias;
 
-    // Received signal strength indicator. Read-only.
-    dbus::Property<int16> rssi;
+    // Received signal strength indicator that is set when the device is
+    // discovered during inquiry. Read-only.
+    dbus::Property<int16> inquiry_rssi;

[keybuk, 2014/05/09 00:35:25]

Properties must match the D-Bus name, keep this as rssi

[Tim Song, 2014/05/09 02:55:29]

Done.

+
+    // Received signal strength indicator when a connection is open to the
+    // device. This property is not set unless connection monitor is enabled.
+    // Read-only.
+    dbus::Property<int16> connection_rssi;
+
+    // The transmit power level of the host when a connection is open
+    // to the device. This property is not set unless connection monitor is
+    // enabled. Read-only.
+    dbus::Property<int16> tx_power;

[keybuk, 2014/05/09 00:35:25]

This is called connection_tx_power in your cros_system_api change. Properties
must match their D-Bus name.

[Tim Song, 2014/05/09 02:55:29]

Done.

+
+    // The maximum transmit power level of the host that can be set
+    // when connected to the device. Read-only.
+    dbus::Property<int16> tx_power_max;

[keybuk, 2014/05/09 00:35:25]

ditto

[Tim Song, 2014/05/09 02:55:29]

Done.

 
     Properties(dbus::ObjectProxy* object_proxy,
                const std::string& interface_name,
                const PropertyChangedCallback& callback);
     virtual ~Properties();
   };
 
   // Interface for observing changes from a remote bluetooth device.
   class Observer {
    public:
@@ skipping 71 matching lines @@
   virtual void Pair(const dbus::ObjectPath& object_path,
                     const base::Closure& callback,
                     const ErrorCallback& error_callback) = 0;
 
   // Cancels an in-progress pairing with the device with object path
   // |object_path| initiated by Pair().
   virtual void CancelPairing(const dbus::ObjectPath& object_path,
                              const base::Closure& callback,
                              const ErrorCallback& error_callback) = 0;
 
+  // Starts connection monitor for the device with object path
+  // |object_path|. Connection monitor is a mode the connection properties,
+  // RSSI and TX power are tracked and updated when they change.
+  virtual void StartConnectionMonitor(const dbus::ObjectPath& object_path,
+                                      const base::Closure& callback,
+                                      const ErrorCallback& error_callback) = 0;
+
+  // Stops connection monitor for the device with object path
+  // |object_path|.
+  virtual void StopConnectionMonitor(const dbus::ObjectPath& object_path,
+                                     const base::Closure& callback,
+                                     const ErrorCallback& error_callback) = 0;
+
   // Creates the instance.
   static BluetoothDeviceClient* Create();
 
   // Constants used to indicate exceptional error conditions.
   static const char kNoResponseError[];
   static const char kUnknownDeviceError[];
 
  protected:
   BluetoothDeviceClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BluetoothDeviceClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_BLUETOOTH_DEVICE_CLIENT_H_