summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author jackqdyulei <jackqdyulei@google.com> 2018-08-20 14:02:18 -0700
committer jackqdyulei <jackqdyulei@google.com> 2018-08-20 15:33:51 -0700
commitcc2294959d85cab3493575fb4ceab5dbb48af0d6 (patch)
treeae7bea4345354672eea4aaf5fa58468ff233f7db
parent6641ac548a389f4b6c329f72081d6a9db16b0a28 (diff)
Remove obsolete code in BluetoothEventMangaer
1. Remove readPairedDevices() because we already did it in constructor. There is no need to call it each time there is a device miss. 2. Remove dispatchDeviceAdd() since it is already been called in addDevice(). Bug: 111803044 Test: test still pass Change-Id: Ib69daf47b1227d31bf1e6ac17a7e0505046adf17
-rw-r--r--packages/SettingsLib/src/com/android/settingslib/bluetooth/BluetoothEventManager.java18
1 files changed, 4 insertions, 14 deletions
diff --git a/packages/SettingsLib/src/com/android/settingslib/bluetooth/BluetoothEventManager.java b/packages/SettingsLib/src/com/android/settingslib/bluetooth/BluetoothEventManager.java
index 466d02bd3bcf..7299c5b214e0 100644
--- a/packages/SettingsLib/src/com/android/settingslib/bluetooth/BluetoothEventManager.java
+++ b/packages/SettingsLib/src/com/android/settingslib/bluetooth/BluetoothEventManager.java
@@ -150,8 +150,7 @@ public class BluetoothEventManager {
for (BluetoothDevice device : bondedDevices) {
CachedBluetoothDevice cachedDevice = mDeviceManager.findDevice(device);
if (cachedDevice == null) {
- cachedDevice = mDeviceManager.addDevice(device);
- dispatchDeviceAdded(cachedDevice);
+ mDeviceManager.addDevice(device);
deviceAdded = true;
}
}
@@ -339,18 +338,9 @@ public class BluetoothEventManager {
BluetoothDevice.ERROR);
CachedBluetoothDevice cachedDevice = mDeviceManager.findDevice(device);
if (cachedDevice == null) {
- Log.w(TAG, "CachedBluetoothDevice for device " + device +
- " not found, calling readPairedDevices().");
- if (readPairedDevices()) {
- cachedDevice = mDeviceManager.findDevice(device);
- }
-
- if (cachedDevice == null) {
- Log.w(TAG, "Got bonding state changed for " + device +
- ", but we have no record of that device.");
- cachedDevice = mDeviceManager.addDevice(device);
- dispatchDeviceAdded(cachedDevice);
- }
+ Log.w(TAG, "Got bonding state changed for " + device +
+ ", but we have no record of that device.");
+ cachedDevice = mDeviceManager.addDevice(device);
}
synchronized (mCallbacks) {