bluetooth: android: Implement RetrieveGattConnectedDevicesWithFilter

device/bluetooth/android/java/src/org/chromium/device/bluetooth/BluetoothDeviceWrapper.java

Index: device/bluetooth/android/java/src/org/chromium/device/bluetooth/BluetoothDeviceWrapper.java
diff --git a/device/bluetooth/android/java/src/org/chromium/device/bluetooth/BluetoothDeviceWrapper.java b/device/bluetooth/android/java/src/org/chromium/device/bluetooth/BluetoothDeviceWrapper.java
new file mode 100644
index 0000000000000000000000000000000000000000..39cbfcc80a9550a0a894d98c5da20321f0a8b002
--- /dev/null
+++ b/device/bluetooth/android/java/src/org/chromium/device/bluetooth/BluetoothDeviceWrapper.java
@@ -0,0 +1,77 @@
+// Copyright 2016 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.
+
+package org.chromium.device.bluetooth;
+
+import android.annotation.TargetApi;
+import android.bluetooth.BluetoothDevice;
+import android.bluetooth.BluetoothGattCharacteristic;
+import android.bluetooth.BluetoothGattDescriptor;
+import android.content.Context;
+import android.os.Build;
+import android.os.ParcelUuid;
+
+import org.chromium.base.annotations.JNINamespace;
+
+import java.util.HashMap;
+
+/**
+ * Wraps android.bluetooth.BluetoothDevice. See Wrappers.java for the rest of the wrappers.
+ * BluetoothDeviceWrapper is in its own file because jni cannot direclty use the nested class

[scheib, 2016/11/28 20:42:22]

Wrapper.java explains what the concept of a wrapper class is, maybe point more
directly to top of Wrappers.java for explanation of this class purpose.

When explaining why it's in its own file, cite http://crbug.com/505554 I can see
this being nice because you got tired of copy pasting the previous comments
explaining the use of 'Object' type, but does it fix anything more than that?

Probably we should explain at top of wrappers.java why there's this terribly
large file and everyhting isn't just split out: doing so would require a .h and
.cpp file for every split .java file that uses JNI. Or, maybe we should just
split out everything ;P.

+ * for:
+ *   @CalledByNative
+ *   private DeviceAndUUIDs[] retrieveGattConnectedDevices() {
+ */
+@JNINamespace("device")

[scheib, 2016/11/28 20:42:22]

This class needs JNINamespace because other classes from that namespace are
using it? It doesn't seem to be called directly by JNI.

+@TargetApi(Build.VERSION_CODES.LOLLIPOP)
+class BluetoothDeviceWrapper {

[scheib, 2016/11/28 20:42:22]

Can this still be "static class"?

+    public static final int DEVICE_CLASS_UNSPECIFIED = 0x1F00;
+
+    private final BluetoothDevice mDevice;
+    final HashMap<BluetoothGattCharacteristic, Wrappers.BluetoothGattCharacteristicWrapper>
+            mCharacteristicsToWrappers;
+    final HashMap<BluetoothGattDescriptor, Wrappers.BluetoothGattDescriptorWrapper>
+            mDescriptorsToWrappers;
+
+    public BluetoothDeviceWrapper(BluetoothDevice device) {
+        mDevice = device;
+        mCharacteristicsToWrappers = new HashMap<BluetoothGattCharacteristic,
+                Wrappers.BluetoothGattCharacteristicWrapper>();
+        mDescriptorsToWrappers =
+                new HashMap<BluetoothGattDescriptor, Wrappers.BluetoothGattDescriptorWrapper>();
+    }
+
+    public Wrappers.BluetoothGattWrapper connectGatt(
+            Context context, boolean autoConnect, Wrappers.BluetoothGattCallbackWrapper callback) {
+        return new Wrappers.BluetoothGattWrapper(
+                mDevice.connectGatt(context, autoConnect,
+                        new Wrappers.ForwardBluetoothGattCallbackToWrapper(callback, this)),
+                this);
+    }
+
+    public String getAddress() {
+        return mDevice.getAddress();
+    }
+
+    public int getBluetoothClass_getDeviceClass() {
+        if (mDevice == null || mDevice.getBluetoothClass() == null) {
+            // BluetoothDevice.getBluetoothClass() returns null if adapter has been powered off.
+            // Return DEVICE_CLASS_UNSPECIFIED in these cases.
+            return DEVICE_CLASS_UNSPECIFIED;
+        }
+        return mDevice.getBluetoothClass().getDeviceClass();
+    }
+
+    public int getBondState() {
+        return mDevice.getBondState();
+    }
+
+    public String getName() {
+        return mDevice.getName();
+    }
+
+    public ParcelUuid[] getUuids() {
+        return mDevice.getUuids();
+    }
+}