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

Commit bd6b5e36 authored by Automerger Merge Worker's avatar Automerger Merge Worker
Browse files

Merge "Add missing RequiresPermission annotations in BluetoothHidHost and... 

Merge "Add missing RequiresPermission annotations in BluetoothHidHost and BluetoothMap APIs, disallow null device input for setConnectionPolicy, getConnectionPolicy, and getConnectionState in BluetoothHidHost, and BluetoothMap implements AutoCloseable, its close() method is public, and utilizes a CloseGuard." am: a9e6a280 am: 7e3f076b am: c4721f1f

Change-Id: I749db7b515ee78cb7d12dcbbaf631bb456114834
parents 49fe1d5e c4721f1f

api/system-current.txt

+8 −6
Original line number Diff line number Diff line
@@ -1633,15 +1633,17 @@ package android.bluetooth { 
  }















  public final class BluetoothHidHost implements android.bluetooth.BluetoothProfile {













    method @NonNull public java.util.List<android.bluetooth.BluetoothDevice> getConnectedDevices();













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH) public int getConnectionPolicy(@Nullable android.bluetooth.BluetoothDevice);













    method public int getConnectionState(@Nullable android.bluetooth.BluetoothDevice);













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_ADMIN) public boolean setConnectionPolicy(@Nullable android.bluetooth.BluetoothDevice, int);













    method @NonNull @RequiresPermission(android.Manifest.permission.BLUETOOTH_PRIVILEGED) public java.util.List<android.bluetooth.BluetoothDevice> getConnectedDevices();













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH) public int getConnectionPolicy(@NonNull android.bluetooth.BluetoothDevice);













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_PRIVILEGED) public int getConnectionState(@NonNull android.bluetooth.BluetoothDevice);













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_ADMIN) public boolean setConnectionPolicy(@NonNull android.bluetooth.BluetoothDevice, int);















    field public static final String ACTION_CONNECTION_STATE_CHANGED = "android.bluetooth.input.profile.action.CONNECTION_STATE_CHANGED";















  }



























  public final class BluetoothMap implements android.bluetooth.BluetoothProfile {













    method @NonNull public java.util.List<android.bluetooth.BluetoothDevice> getConnectedDevices();













  public final class BluetoothMap implements java.lang.AutoCloseable android.bluetooth.BluetoothProfile {













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_PRIVILEGED) public void close();













    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_PRIVILEGED) protected void finalize();













    method @NonNull @RequiresPermission(android.Manifest.permission.BLUETOOTH_PRIVILEGED) public java.util.List<android.bluetooth.BluetoothDevice> getConnectedDevices();















    method @RequiresPermission(android.Manifest.permission.BLUETOOTH) public int getConnectionPolicy(@Nullable android.bluetooth.BluetoothDevice);















    method @RequiresPermission(android.Manifest.permission.BLUETOOTH_ADMIN) public boolean setConnectionPolicy(@Nullable android.bluetooth.BluetoothDevice, int);















    field public static final String ACTION_CONNECTION_STATE_CHANGED = "android.bluetooth.map.profile.action.CONNECTION_STATE_CHANGED";

































core/java/android/bluetooth/BluetoothHidHost.java



+14
−4




























Original line number
Diff line number
Diff line








@@ -18,7 +18,6 @@ package android.bluetooth;





























import android.Manifest;















import android.annotation.NonNull;













import android.annotation.Nullable;















import android.annotation.RequiresPermission;















import android.annotation.SdkConstant;















import android.annotation.SdkConstant.SdkConstantType;










@@ -330,6 +329,7 @@ public final class BluetoothHidHost implements BluetoothProfile {













     * {@inheritDoc}















     */















    @Override













    @RequiresPermission(Manifest.permission.BLUETOOTH_PRIVILEGED)















    public @NonNull List<BluetoothDevice> getConnectedDevices() {















        if (VDBG) log("getConnectedDevices()");















        final IBluetoothHidHost service = getService();










@@ -370,8 +370,12 @@ public final class BluetoothHidHost implements BluetoothProfile {













     * {@inheritDoc}















     */















    @Override













    public int getConnectionState(@Nullable BluetoothDevice device) {













    @RequiresPermission(Manifest.permission.BLUETOOTH_PRIVILEGED)













    public int getConnectionState(@NonNull BluetoothDevice device) {















        if (VDBG) log("getState(" + device + ")");













        if (device == null) {













            throw new IllegalArgumentException("device must not be null");













        }















        final IBluetoothHidHost service = getService();















        if (service != null && isEnabled() && isValidDevice(device)) {















            try {










@@ -416,9 +420,12 @@ public final class BluetoothHidHost implements BluetoothProfile {













     */















    @SystemApi















    @RequiresPermission(Manifest.permission.BLUETOOTH_ADMIN)













    public boolean setConnectionPolicy(@Nullable BluetoothDevice device,













    public boolean setConnectionPolicy(@NonNull BluetoothDevice device,















            @ConnectionPolicy int connectionPolicy) {















        if (DBG) log("setConnectionPolicy(" + device + ", " + connectionPolicy + ")");













        if (device == null) {













            throw new IllegalArgumentException("device must not be null");













        }















        final IBluetoothHidHost service = getService();















        if (service != null && isEnabled() && isValidDevice(device)) {















            if (connectionPolicy != BluetoothProfile.CONNECTION_POLICY_FORBIDDEN










@@ -465,8 +472,11 @@ public final class BluetoothHidHost implements BluetoothProfile {













     */















    @SystemApi















    @RequiresPermission(Manifest.permission.BLUETOOTH)













    public @ConnectionPolicy int getConnectionPolicy(@Nullable BluetoothDevice device) {













    public @ConnectionPolicy int getConnectionPolicy(@NonNull BluetoothDevice device) {















        if (VDBG) log("getConnectionPolicy(" + device + ")");













        if (device == null) {













            throw new IllegalArgumentException("device must not be null");













        }















        final IBluetoothHidHost service = getService();















        if (service != null && isEnabled() && isValidDevice(device)) {















            try {

































core/java/android/bluetooth/BluetoothMap.java



+16
−8




























Original line number
Diff line number
Diff line








@@ -27,6 +27,7 @@ import android.content.Context;













import android.os.Binder;















import android.os.IBinder;















import android.os.RemoteException;













import android.util.CloseGuard;















import android.util.Log;































import java.util.ArrayList;









@@ -39,12 +40,14 @@ import java.util.List;













 * @hide















 */















@SystemApi













public final class BluetoothMap implements BluetoothProfile {













public final class BluetoothMap implements BluetoothProfile, AutoCloseable {































    private static final String TAG = "BluetoothMap";















    private static final boolean DBG = true;















    private static final boolean VDBG = false;





























    private CloseGuard mCloseGuard;





























    /** @hide */















    @SuppressLint("ActionValue")















    @SystemApi










@@ -86,15 +89,16 @@ public final class BluetoothMap implements BluetoothProfile {













        if (DBG) Log.d(TAG, "Create BluetoothMap proxy object");















        mAdapter = BluetoothAdapter.getDefaultAdapter();















        mProfileConnector.connect(context, listener);













        mCloseGuard = new CloseGuard();













        mCloseGuard.open("close");















    }





























    @SuppressLint("GenericException")













    protected void finalize() throws Throwable {













        try {













            close();













        } finally {













            super.finalize();













    @RequiresPermission(Manifest.permission.BLUETOOTH_PRIVILEGED)













    protected void finalize() {













        if (mCloseGuard != null) {













            mCloseGuard.warnIfOpen();















        }













        close();















    }































    /**









@@ -105,7 +109,10 @@ public final class BluetoothMap implements BluetoothProfile {













     *















     * @hide















     */













    public synchronized void close() {













    @SystemApi













    @RequiresPermission(Manifest.permission.BLUETOOTH_PRIVILEGED)













    public void close() {













        if (VDBG) log("close()");















        mProfileConnector.disconnect();















    }


























@@ -250,6 +257,7 @@ public final class BluetoothMap implements BluetoothProfile {













     * @hide















     */















    @SystemApi













    @RequiresPermission(Manifest.permission.BLUETOOTH_PRIVILEGED)















    public @NonNull List<BluetoothDevice> getConnectedDevices() {















        if (DBG) log("getConnectedDevices()");















        final IBluetoothMap service = getService();