bluetooth: Create base class BluetoothDevice::CreateGattConnection impl.

device/bluetooth/bluetooth_gatt_connection.h

 // 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.
 
 #ifndef DEVICE_BLUETOOTH_BLUETOOTH_GATT_CONNECTION_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_GATT_CONNECTION_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "device/bluetooth/bluetooth_export.h"
 
 namespace device {
 
+class BluetoothAdapter;
+
 // BluetoothGattConnection represents a GATT connection to a Bluetooth device
 // that has GATT services. Instances are obtained from a BluetoothDevice,
 // and the connection is kept alive as long as there is at least one
 // active BluetoothGattConnection object. BluetoothGattConnection objects
 // automatically update themselves, when the connection is terminated by the
 // operating system (e.g. due to user action).
 class DEVICE_BLUETOOTH_EXPORT BluetoothGattConnection {
  public:
+  BluetoothGattConnection(BluetoothAdapter* adapter,

[Jeffrey Yasskin, 2015/08/19 22:49:46]

Take this as a scoped_refptr<> to make it clear that you're keeping a reference.

[scheib, 2015/09/13 02:40:28]

Done.

+                          const std::string& device_address);
+
   // Destructor automatically closes this GATT connection. If this is the last
   // remaining GATT connection and this results in a call to the OS, that call
   // may not always succeed. Users can make an explicit call to
   // BluetoothGattConnection::Close to make sure that they are notified of
   // a possible error via the callback.
   virtual ~BluetoothGattConnection();
 
   // Returns the Bluetooth address of the device that this connection is open
   // to.
-  virtual std::string GetDeviceAddress() const = 0;
+  std::string GetDeviceAddress() const;
 
-  // Returns true if this connection is open.
-  virtual bool IsConnected() = 0;
+  // Returns true if this GATT connection is open.
+  virtual bool IsConnected();
 
   // Disconnects this GATT connection. The device may still remain connected due
   // to other GATT connections.
-  virtual void Disconnect() = 0;
+  virtual void Disconnect();
 
  protected:
-  BluetoothGattConnection();
+  // The Bluetooth adapter that this connection is associated with. A reference
+  // is held because BluetoothGattConnection keeps the connection alive.
+  scoped_refptr<BluetoothAdapter> adapter_;

[Jeffrey Yasskin, 2015/08/19 22:49:46]

I'm nervous about holding a reference to BluetoothAdapter from an object that's
notionally "inside" the adapter. How do we systematically prevent reference
cycles?

[scheib, 2015/09/13 02:40:28]

The adapter doesn't own the connections. The connections are vended to the
consumers of the adapter. Holding a connection object keeps the connection
alive. We must keep the adapter alive to keep the connection alive. This design
predates this patch set and is how ChromeOS is implemented
 src/device/bluetooth/bluetooth_gatt_connection_chromeos.h

https://code.google.com/p/chromium/codesearch#chromium/src/device/bluetooth/b...

+
+  // Bluetooth address of the underlying device.
+  std::string device_address_;
 
  private:
+  bool already_decremented_connection_reference_on_device_ = false;

[Jeffrey Yasskin, 2015/08/19 22:49:46]

Call this something like "owns_reference_for_connection_" or even just
"connected_". The device could be connected without a particular Connection
handle being connected.

[scheib, 2015/09/13 02:40:28]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(BluetoothGattConnection);
 };
 
 }  // namespace device
 
 #endif  //  DEVICE_BLUETOOTH_BLUETOOTH_GATT_CONNECTION_H_