From 50c064d5025ac8634c05959addf1629e9b8d32e6 Mon Sep 17 00:00:00 2001 From: hughchen Date: Tue, 14 Jan 2020 16:07:59 +0800 Subject: [PATCH] Update api name Use CachedBluetoothDevice.connect() instead of CachedBluetoothDevice.connect(boolean) to connect Bluetooth device. Fixes: 141582844 Test: manually Change-Id: I8c50678bdfd6de89fb8d56122bfa38bdd5ac5c9d Merged-In: I8c50678bdfd6de89fb8d56122bfa38bdd5ac5c9d --- .../bluetooth/BluetoothDetailsButtonsController.java | 2 +- .../settings/bluetooth/BluetoothDevicePreference.java | 2 +- .../settings/bluetooth/SavedBluetoothDeviceUpdater.java | 2 +- .../bluetooth/BluetoothDetailsButtonsControllerTest.java | 5 ++--- .../settings/bluetooth/SavedBluetoothDeviceUpdaterTest.java | 2 +- 5 files changed, 6 insertions(+), 7 deletions(-) diff --git a/src/com/android/settings/bluetooth/BluetoothDetailsButtonsController.java b/src/com/android/settings/bluetooth/BluetoothDetailsButtonsController.java index a7fae1441f9..c8078938b6d 100644 --- a/src/com/android/settings/bluetooth/BluetoothDetailsButtonsController.java +++ b/src/com/android/settings/bluetooth/BluetoothDetailsButtonsController.java @@ -79,7 +79,7 @@ public class BluetoothDetailsButtonsController extends BluetoothDetailsControlle .setButton2Text(R.string.bluetooth_device_context_connect) .setButton2Icon(R.drawable.ic_add_24dp) .setButton2OnClickListener( - view -> mCachedDevice.connect(true /* connectAllProfiles */)); + view -> mCachedDevice.connect()); mConnectButtonInitialized = true; } } diff --git a/src/com/android/settings/bluetooth/BluetoothDevicePreference.java b/src/com/android/settings/bluetooth/BluetoothDevicePreference.java index 74d3b6a0594..919dc5b6c87 100644 --- a/src/com/android/settings/bluetooth/BluetoothDevicePreference.java +++ b/src/com/android/settings/bluetooth/BluetoothDevicePreference.java @@ -218,7 +218,7 @@ public final class BluetoothDevicePreference extends GearPreference implements } else if (bondState == BluetoothDevice.BOND_BONDED) { metricsFeatureProvider.action(context, SettingsEnums.ACTION_SETTINGS_BLUETOOTH_CONNECT); - mCachedDevice.connect(true); + mCachedDevice.connect(); } else if (bondState == BluetoothDevice.BOND_NONE) { metricsFeatureProvider.action(context, SettingsEnums.ACTION_SETTINGS_BLUETOOTH_PAIR); diff --git a/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdater.java b/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdater.java index 6381b848c9c..bcaf38578a7 100644 --- a/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdater.java +++ b/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdater.java @@ -53,7 +53,7 @@ public class SavedBluetoothDeviceUpdater extends BluetoothDeviceUpdater public boolean onPreferenceClick(Preference preference) { final CachedBluetoothDevice device = ((BluetoothDevicePreference) preference) .getBluetoothDevice(); - device.connect(true); + device.connect(); return true; } } diff --git a/tests/robotests/src/com/android/settings/bluetooth/BluetoothDetailsButtonsControllerTest.java b/tests/robotests/src/com/android/settings/bluetooth/BluetoothDetailsButtonsControllerTest.java index eaa4c3cc968..f52685c3ec1 100644 --- a/tests/robotests/src/com/android/settings/bluetooth/BluetoothDetailsButtonsControllerTest.java +++ b/tests/robotests/src/com/android/settings/bluetooth/BluetoothDetailsButtonsControllerTest.java @@ -22,7 +22,6 @@ import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.anyBoolean; import static org.mockito.ArgumentMatchers.anyInt; import static org.mockito.ArgumentMatchers.anyString; -import static org.mockito.ArgumentMatchers.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -103,7 +102,7 @@ public class BluetoothDetailsButtonsControllerTest extends BluetoothDetailsContr verify(mButtonsPref).setButton2Text(R.string.bluetooth_device_context_connect); mConnectButton.callOnClick(); - verify(mCachedDevice).connect(eq(true)); + verify(mCachedDevice).connect(); } @Test @@ -119,7 +118,7 @@ public class BluetoothDetailsButtonsControllerTest extends BluetoothDetailsContr // Click the button and make sure that connect (not disconnect) gets called. mConnectButton.callOnClick(); - verify(mCachedDevice).connect(eq(true)); + verify(mCachedDevice).connect(); } @Test diff --git a/tests/robotests/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdaterTest.java b/tests/robotests/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdaterTest.java index ac7692d470a..89604a92c2c 100644 --- a/tests/robotests/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdaterTest.java +++ b/tests/robotests/src/com/android/settings/bluetooth/SavedBluetoothDeviceUpdaterTest.java @@ -121,6 +121,6 @@ public class SavedBluetoothDeviceUpdaterTest { public void onClick_Preference_setConnect() { mBluetoothDeviceUpdater.onPreferenceClick(mPreference); - verify(mCachedBluetoothDevice).connect(true); + verify(mCachedBluetoothDevice).connect(); } }