Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f33beead authored by William Escande's avatar William Escande
Browse files

Always return nonNull in BluetoothAdapter.getUuids 

Test: atest CtsBluetoothTestCases
Fix: 221850407
Tag: #refactor
Change-Id: Iaf312835ca0731b5b8ae0b9f6ea86ed67dc78321
parent b6e8f65a

framework/api/system-current.txt

+1 −1
Original line number Diff line number Diff line
@@ -65,7 +65,7 @@ package android.bluetooth { 
    method @RequiresPermission(allOf={android.Manifest.permission.BLUETOOTH_CONNECT, android.Manifest.permission.BLUETOOTH_PRIVILEGED}) public long getDiscoveryEndMillis();

    method @NonNull @RequiresPermission(allOf={android.Manifest.permission.BLUETOOTH_CONNECT, android.Manifest.permission.BLUETOOTH_PRIVILEGED}) public java.util.List<android.bluetooth.BluetoothDevice> getMostRecentlyConnectedDevices();

    method @NonNull @RequiresPermission(allOf={android.Manifest.permission.BLUETOOTH_CONNECT, android.Manifest.permission.BLUETOOTH_PRIVILEGED}) public java.util.List<java.lang.Integer> getSupportedProfiles();

    method @Nullable @RequiresPermission(android.Manifest.permission.BLUETOOTH_CONNECT) public android.os.ParcelUuid[] getUuids();

    method @NonNull @RequiresPermission(android.Manifest.permission.BLUETOOTH_CONNECT) public android.os.ParcelUuid[] getUuids();

    method public boolean isBleScanAlwaysAvailable();

    method public boolean isLeEnabled();

    method @NonNull public static String nameForState(int);

framework/java/android/bluetooth/BluetoothAdapter.java

+3 −3
Original line number Diff line number Diff line
@@ -1438,9 +1438,9 @@ public final class BluetoothAdapter { 
    @RequiresBluetoothConnectPermission

    @RequiresPermission(android.Manifest.permission.BLUETOOTH_CONNECT)

    @SuppressLint(value = {"ArrayReturn", "NullableCollection"})

    public @Nullable ParcelUuid[] getUuids() {

    public @NonNull ParcelUuid[] getUuids() {

        if (getState() != STATE_ON) {

            return null;

            return new ParcelUuid[0];

        }

        try {

            mServiceLock.readLock().lock();
@@ -1457,7 +1457,7 @@ public final class BluetoothAdapter { 
        } finally {

            mServiceLock.readLock().unlock();

        }

        return null;

        return new ParcelUuid[0];

    }



    /**