aljaz/Android-nRF-Toolbox
1
0
Fork 0
mirror of https://github.com/aljazceru/Android-nRF-Toolbox.git synced 2026-01-09 17:54:27 +01:00
Code Issues Packages Projects Releases Wiki Activity

Recent changes in BleManager copied to wear/BleManager

Browse Source
This commit is contained in:
Aleksander Nowakowski
2017-03-15 11:25:28 +01:00
parent 6b5b66b123
commit 38546cf4a0
2 changed files with 36 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/src/main/java/no/nordicsemi/android/nrftoolbox/profile/BleManager.java
View File

@@ -1149,21 +1149,19 @@ public abstract class BleManager<E extends BleManagerCallbacks> implements ILogg
* Our tests has shown that 600 ms is enough(*). It is important to call it AFTER receiving the SC indication, but not necessarily
* after Android finishes the internal service discovery.
*
* NOTE: This applies only for bonded devices with Service Changed characteristic, but to be sure we will postpone
* service discovery for all devices.
*
* (*) - While testing on Nexus 4 with Android 5.0.1 it appeared that 600 ms may be not enough.
* The service discovery initiated by the system was still in progress while gatt.serviceDiscovery() below
* was called and (due to a bug on older Android versions, where there can be only one callback registered)
* the onServicesDiscovered(...) callback has never been called.
*/
// On Android Nougat or never devices no delay is necessary any more.
final boolean bonded = gatt.getDevice().getBondState() == BluetoothDevice.BOND_BONDED;
int delay = 0;
// TODO: modify the delay to match your use case. More services require longer discovery time. Use Nexus 4, 7 or other phone with bad throughput.
// The delays here are based on a guess. Newer phones should handle service discovery it faster, I guess.
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.N)
if (bonded && Build.VERSION.SDK_INT < Build.VERSION_CODES.N)
delay = 600;
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M)
if (bonded && Build.VERSION.SDK_INT < Build.VERSION_CODES.M)
delay = 1200;
if (delay > 0)
Logger.d(mLogSession, "wait(" + delay + ")");

43
wear/src/main/java/no/nordicsemi/android/nrftoolbox/ble/BleManager.java
View File

@@ -103,8 +103,8 @@ public class BleManager implements BleProfileApi {
if (mConnected && previousState != BluetoothAdapter.STATE_TURNING_OFF && previousState != BluetoothAdapter.STATE_OFF) {
// The connection is killed by the system, no need to gently disconnect
mGattCallback.notifyDeviceDisconnected(mBluetoothDevice);
close();
}
close();
break;
}
}
@@ -193,8 +193,30 @@ public class BleManager implements BleProfileApi {
synchronized (mLock) {
if (mBluetoothGatt != null) {
mBluetoothGatt.close();
mBluetoothGatt = null;
// There are 2 ways of reconnecting to the same device:
// 1. Reusing the same BluetoothGatt object and calling connect() on it or
// 2. Closing it and reopening a new instance of BluetoothGatt object.
// The gatt.close() is an asynchronous method. It requires some time before it's finished and
// device.connectGatt(...) can't be called immediately or service discovery
// may never finish on some older devices (Nexus 4, Android 5.0.1).
// However, the autoConnect flag settings may have changed. If it did, try closing and opening new BluetoothGatt
// despite the difficulties.
final boolean autoConnect = shouldAutoConnect();
if (autoConnect == mUserDisconnected) { // autoConnect has changed. Previously mUserDisconnected was set to !autoConnect (see below)
mBluetoothGatt.close();
mBluetoothGatt = null;
try {
Thread.sleep(200); // Is 200 ms enough?
} catch (final InterruptedException e) {
// Ignore
}
} else {
// Instead, the gatt.connect() method will be used to reconnect to the same device.
mConnectionState = BluetoothGatt.STATE_CONNECTING;
mCallbacks.onDeviceConnecting(device);
mBluetoothGatt.connect();
return;
}
} else {
// Register bonding broadcast receiver
mContext.registerReceiver(mBluetoothStateBroadcastReceiver, new IntentFilter(BluetoothAdapter.ACTION_STATE_CHANGED));
@@ -322,7 +344,7 @@ public class BleManager implements BleProfileApi {
if (scCharacteristic == null)
return false;
return enableIndications(scCharacteristic);
return internalEnableIndications(scCharacteristic);
}
@Override
@@ -531,7 +553,7 @@ public class BleManager implements BleProfileApi {
private final Queue<Request> mTaskQueue = new LinkedList<>();
private Deque<Request> mInitQueue;
private boolean mInitInProgress;
private boolean mOperationInProgress;
private boolean mOperationInProgress = true;
private void notifyDeviceDisconnected(final BluetoothDevice device) {
mConnected = false;
@@ -574,10 +596,9 @@ public class BleManager implements BleProfileApi {
* Therefore we have to postpone the service discovery operation until we are (almost, as there is no such callback) sure, that it had to be handled.
* Our tests has shown that 600 ms is enough. It is important to call it AFTER receiving the SC indication, but not necessarily
* after Android finishes the internal service discovery.
*
* NOTE: This applies only for bonded devices with Service Changed characteristic, but to be sure we will postpone
* service discovery for all devices.
*/
final boolean bonded = gatt.getDevice().getBondState() == BluetoothDevice.BOND_BONDED;
int delay = bonded ? 600 : 0;
mHandler.postDelayed(new Runnable() {
@Override
public void run() {
@@ -586,11 +607,13 @@ public class BleManager implements BleProfileApi {
gatt.discoverServices();
}
}
}, 600);
}, delay);
} else {
if (newState == BluetoothProfile.STATE_DISCONNECTED) {
mOperationInProgress = true; // no more calls are possible
notifyDeviceDisconnected(gatt.getDevice());
if (mConnected) {
notifyDeviceDisconnected(gatt.getDevice());
}
return;
}