bluetooth: android: Initial basic Descriptors implementation.

device/bluetooth/bluetooth_remote_gatt_characteristic_android.h

 // Copyright 2015 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_REMOTE_GATT_CHARACTERISTIC_ANDROID_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_REMOTE_GATT_CHARACTERISTIC_ANDROID_H_
 
 #include <stdint.h>
 
 #include "base/android/jni_android.h"
+#include "base/containers/scoped_ptr_hash_map.h"
 #include "base/macros.h"
 #include "device/bluetooth/bluetooth_gatt_characteristic.h"
 
 namespace device {
 
+class BluetoothAdapterAndroid;
+class BluetoothRemoteGattDescriptorAndroid;
+
 // BluetoothRemoteGattCharacteristicAndroid along with its owned Java class
 // org.chromium.device.bluetooth.ChromeBluetoothRemoteGattCharacteristic
 // implement BluetootGattCharacteristic.
 //
 // TODO(crbug.com/551634): When notifications are enabled characteristic updates
 // should call observers' GattCharacteristicValueChanged.
 class DEVICE_BLUETOOTH_EXPORT BluetoothRemoteGattCharacteristicAndroid
     : public BluetoothGattCharacteristic {
  public:
   // Create a BluetoothRemoteGattCharacteristicAndroid instance and associated
   // Java
   // ChromeBluetoothRemoteGattCharacteristic using the provided
   // |bluetooth_gatt_characteristic_wrapper|.
   //
   // The ChromeBluetoothRemoteGattCharacteristic instance will hold a Java
   // reference
   // to |bluetooth_gatt_characteristic_wrapper|.
   static scoped_ptr<BluetoothRemoteGattCharacteristicAndroid> Create(
+      BluetoothAdapterAndroid* adapter,
       const std::string& instance_id,
       jobject /* BluetoothGattCharacteristicWrapper */
       bluetooth_gatt_characteristic_wrapper,
       jobject /* ChromeBluetoothDevice */ chrome_bluetooth_device);
 
   ~BluetoothRemoteGattCharacteristicAndroid() override;
 
   // Register C++ methods exposed to Java using JNI.
   static bool RegisterJNI(JNIEnv* env);
 
@@ skipping 26 matching lines @@
   void OnRead(JNIEnv* env,
               const base::android::JavaParamRef<jobject>& jcaller,
               int32_t status,
               const base::android::JavaParamRef<jbyteArray>& value);
 
   // Callback after Write operation completes.
   void OnWrite(JNIEnv* env,
                const base::android::JavaParamRef<jobject>& jcaller,
                int32_t status);
 
+  // Creates a Bluetooth GATT descriptor object and adds it to |descriptors_|

[Jeffrey Yasskin, 2016/01/12 21:53:54]

Add a comma before 'DCHECKing'.

[scheib, 2016/01/12 23:16:56]

Done.

+  // DCHECKing that it has not already been created.
+  void CreateGattRemoteDescriptor(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& caller,
+      const base::android::JavaParamRef<jstring>& instanceId,
+      const base::android::JavaParamRef<
+          jobject>& /* BluetoothGattDescriptorWrapper */
+      bluetooth_gatt_descriptor_wrapper,
+      const base::android::JavaParamRef<
+          jobject>& /* ChromeBluetoothCharacteristic */
+      chrome_bluetooth_characteristic);
+
  private:
-  BluetoothRemoteGattCharacteristicAndroid(const std::string& instance_id);
+  BluetoothRemoteGattCharacteristicAndroid(BluetoothAdapterAndroid* adapter,
+                                           const std::string& instance_id);
+
+  // Populates |descriptors_| from Java objects if necessary.
+  void EnsureDescriptorsCreated() const;
+
+  // The adapter associated with this service. It's ok to store a raw pointer
+  // here since |adapter_| indirectly owns this instance.
+  BluetoothAdapterAndroid* adapter_;
 
   // Java object
   // org.chromium.device.bluetooth.ChromeBluetoothRemoteGattCharacteristic.
   base::android::ScopedJavaGlobalRef<jobject> j_characteristic_;
 
   // Adapter unique instance ID.
   std::string instance_id_;
 
   // ReadRemoteCharacteristic callbacks and pending state.
   bool read_pending_ = false;
   ValueCallback read_callback_;
   ErrorCallback read_error_callback_;
 
   // WriteRemoteCharacteristic callbacks and pending state.
   bool write_pending_ = false;
   base::Closure write_callback_;
   ErrorCallback write_error_callback_;
 
   std::vector<uint8_t> value_;
 
+  // Map of descriptors, keyed by descriptor identifier.
+  base::ScopedPtrHashMap<std::string,
+                         scoped_ptr<BluetoothRemoteGattDescriptorAndroid>>
+      descriptors_;
+
   DISALLOW_COPY_AND_ASSIGN(BluetoothRemoteGattCharacteristicAndroid);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_REMOTE_GATT_CHARACTERISTIC_ANDROID_H_