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

Commit a672ffa4 authored by Jack He's avatar Jack He Committed by android-build-merger
Browse files

Merge "Bluetooth: Thread-safe binder invocation" 

am: 94f1fd0d

Change-Id: I8f9c76996f02c70b8d2ea21d4eb567eaa9da246a
parents 04d061e8 94f1fd0d

core/java/android/bluetooth/BluetoothA2dpSink.java

+40 −41
Original line number Original line Diff line number Diff line
@@ -125,7 +125,7 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 




    private Context mContext;

    private Context mContext;

    private ServiceListener mServiceListener;

    private ServiceListener mServiceListener;

    private IBluetoothA2dpSink mService;

    private volatile IBluetoothA2dpSink mService;

    private BluetoothAdapter mAdapter;

    private BluetoothAdapter mAdapter;





    private final IBluetoothStateChangeCallback mBluetoothStateChangeCallback =

    private final IBluetoothStateChangeCallback mBluetoothStateChangeCallback =
@@ -240,15 +240,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

     */

    public boolean connect(BluetoothDevice device) {

    public boolean connect(BluetoothDevice device) {

        if (DBG) log("connect(" + device + ")");

        if (DBG) log("connect(" + device + ")");

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

        final IBluetoothA2dpSink service = mService;

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

            try {

            try {

                return mService.connect(device);

                return service.connect(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return false;

                return false;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return false;

        return false;

    }

    }
@@ -279,15 +280,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

     */

    public boolean disconnect(BluetoothDevice device) {

    public boolean disconnect(BluetoothDevice device) {

        if (DBG) log("disconnect(" + device + ")");

        if (DBG) log("disconnect(" + device + ")");

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

        final IBluetoothA2dpSink service = mService;

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

            try {

            try {

                return mService.disconnect(device);

                return service.disconnect(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return false;

                return false;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return false;

        return false;

    }

    }
@@ -297,15 +299,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
    @Override

    @Override

    public List<BluetoothDevice> getConnectedDevices() {

    public List<BluetoothDevice> getConnectedDevices() {

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

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

        if (mService != null && isEnabled()) {

        final IBluetoothA2dpSink service = mService;

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

            try {

            try {

                return mService.getConnectedDevices();

                return service.getConnectedDevices();

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return new ArrayList<BluetoothDevice>();

                return new ArrayList<BluetoothDevice>();

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return new ArrayList<BluetoothDevice>();

        return new ArrayList<BluetoothDevice>();

    }

    }
@@ -315,15 +318,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
    @Override

    @Override

    public List<BluetoothDevice> getDevicesMatchingConnectionStates(int[] states) {

    public List<BluetoothDevice> getDevicesMatchingConnectionStates(int[] states) {

        if (VDBG) log("getDevicesMatchingStates()");

        if (VDBG) log("getDevicesMatchingStates()");

        if (mService != null && isEnabled()) {

        final IBluetoothA2dpSink service = mService;

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

            try {

            try {

                return mService.getDevicesMatchingConnectionStates(states);

                return service.getDevicesMatchingConnectionStates(states);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return new ArrayList<BluetoothDevice>();

                return new ArrayList<BluetoothDevice>();

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return new ArrayList<BluetoothDevice>();

        return new ArrayList<BluetoothDevice>();

    }

    }
@@ -333,16 +337,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
    @Override

    @Override

    public int getConnectionState(BluetoothDevice device) {

    public int getConnectionState(BluetoothDevice device) {

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

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

        if (mService != null && isEnabled()

        final IBluetoothA2dpSink service = mService;

                && isValidDevice(device)) {

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

            try {

            try {

                return mService.getConnectionState(device);

                return service.getConnectionState(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return BluetoothProfile.STATE_DISCONNECTED;

                return BluetoothProfile.STATE_DISCONNECTED;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return BluetoothProfile.STATE_DISCONNECTED;

        return BluetoothProfile.STATE_DISCONNECTED;

    }

    }
@@ -359,16 +363,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

     */

    public BluetoothAudioConfig getAudioConfig(BluetoothDevice device) {

    public BluetoothAudioConfig getAudioConfig(BluetoothDevice device) {

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

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

        if (mService != null && isEnabled()

        final IBluetoothA2dpSink service = mService;

                && isValidDevice(device)) {

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

            try {

            try {

                return mService.getAudioConfig(device);

                return service.getAudioConfig(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return null;

                return null;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return null;

        return null;

    }

    }
@@ -389,20 +393,20 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

     */

    public boolean setPriority(BluetoothDevice device, int priority) {

    public boolean setPriority(BluetoothDevice device, int priority) {

        if (DBG) log("setPriority(" + device + ", " + priority + ")");

        if (DBG) log("setPriority(" + device + ", " + priority + ")");

        if (mService != null && isEnabled()

        final IBluetoothA2dpSink service = mService;

                && isValidDevice(device)) {

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

            if (priority != BluetoothProfile.PRIORITY_OFF

            if (priority != BluetoothProfile.PRIORITY_OFF

                    && priority != BluetoothProfile.PRIORITY_ON) {

                    && priority != BluetoothProfile.PRIORITY_ON) {

                return false;

                return false;

            }

            }

            try {

            try {

                return mService.setPriority(device, priority);

                return service.setPriority(device, priority);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return false;

                return false;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return false;

        return false;

    }

    }
@@ -421,16 +425,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

     */

    public int getPriority(BluetoothDevice device) {

    public int getPriority(BluetoothDevice device) {

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

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

        if (mService != null && isEnabled()

        final IBluetoothA2dpSink service = mService;

                && isValidDevice(device)) {

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

            try {

            try {

                return mService.getPriority(device);

                return service.getPriority(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return BluetoothProfile.PRIORITY_OFF;

                return BluetoothProfile.PRIORITY_OFF;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return BluetoothProfile.PRIORITY_OFF;

        return BluetoothProfile.PRIORITY_OFF;

    }

    }
@@ -442,16 +446,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     * @param device BluetoothDevice device

     * @param device BluetoothDevice device

     */

     */

    public boolean isA2dpPlaying(BluetoothDevice device) {

    public boolean isA2dpPlaying(BluetoothDevice device) {

        if (mService != null && isEnabled()

        final IBluetoothA2dpSink service = mService;

                && isValidDevice(device)) {

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

            try {

            try {

                return mService.isA2dpPlaying(device);

                return service.isA2dpPlaying(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return false;

                return false;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return false;

        return false;

    }

    }
@@ -485,7 +489,6 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
        public void onServiceConnected(ComponentName className, IBinder service) {

        public void onServiceConnected(ComponentName className, IBinder service) {

            if (DBG) Log.d(TAG, "Proxy object connected");

            if (DBG) Log.d(TAG, "Proxy object connected");

            mService = IBluetoothA2dpSink.Stub.asInterface(Binder.allowBlocking(service));

            mService = IBluetoothA2dpSink.Stub.asInterface(Binder.allowBlocking(service));



            if (mServiceListener != null) {

            if (mServiceListener != null) {

                mServiceListener.onServiceConnected(BluetoothProfile.A2DP_SINK,

                mServiceListener.onServiceConnected(BluetoothProfile.A2DP_SINK,

                        BluetoothA2dpSink.this);

                        BluetoothA2dpSink.this);
@@ -502,15 +505,11 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
    };

    };





    private boolean isEnabled() {

    private boolean isEnabled() {

        if (mAdapter.getState() == BluetoothAdapter.STATE_ON) return true;

        return mAdapter.getState() == BluetoothAdapter.STATE_ON;

        return false;

    }

    }





    private boolean isValidDevice(BluetoothDevice device) {

    private static boolean isValidDevice(BluetoothDevice device) {

        if (device == null) return false;

        return device != null && BluetoothAdapter.checkBluetoothAddress(device.getAddress());



        if (BluetoothAdapter.checkBluetoothAddress(device.getAddress())) return true;

        return false;

    }

    }





    private static void log(String msg) {

    private static void log(String msg) {

core/java/android/bluetooth/BluetoothAvrcpController.java

+27 −27
Original line number Original line Diff line number Diff line
@@ -81,7 +81,7 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 




    private Context mContext;

    private Context mContext;

    private ServiceListener mServiceListener;

    private ServiceListener mServiceListener;

    private IBluetoothAvrcpController mService;

    private volatile IBluetoothAvrcpController mService;

    private BluetoothAdapter mAdapter;

    private BluetoothAdapter mAdapter;





    private final IBluetoothStateChangeCallback mBluetoothStateChangeCallback =

    private final IBluetoothStateChangeCallback mBluetoothStateChangeCallback =
@@ -179,15 +179,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    @Override

    @Override

    public List<BluetoothDevice> getConnectedDevices() {

    public List<BluetoothDevice> getConnectedDevices() {

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

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

        if (mService != null && isEnabled()) {

        final IBluetoothAvrcpController service = mService;

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

            try {

            try {

                return mService.getConnectedDevices();

                return service.getConnectedDevices();

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return new ArrayList<BluetoothDevice>();

                return new ArrayList<BluetoothDevice>();

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return new ArrayList<BluetoothDevice>();

        return new ArrayList<BluetoothDevice>();

    }

    }
@@ -197,15 +198,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    @Override

    @Override

    public List<BluetoothDevice> getDevicesMatchingConnectionStates(int[] states) {

    public List<BluetoothDevice> getDevicesMatchingConnectionStates(int[] states) {

        if (VDBG) log("getDevicesMatchingStates()");

        if (VDBG) log("getDevicesMatchingStates()");

        if (mService != null && isEnabled()) {

        final IBluetoothAvrcpController service = mService;

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

            try {

            try {

                return mService.getDevicesMatchingConnectionStates(states);

                return service.getDevicesMatchingConnectionStates(states);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return new ArrayList<BluetoothDevice>();

                return new ArrayList<BluetoothDevice>();

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return new ArrayList<BluetoothDevice>();

        return new ArrayList<BluetoothDevice>();

    }

    }
@@ -215,16 +217,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    @Override

    @Override

    public int getConnectionState(BluetoothDevice device) {

    public int getConnectionState(BluetoothDevice device) {

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

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

        if (mService != null && isEnabled()

        final IBluetoothAvrcpController service = mService;

                && isValidDevice(device)) {

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

            try {

            try {

                return mService.getConnectionState(device);

                return service.getConnectionState(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                Log.e(TAG, "Stack:" + Log.getStackTraceString(new Throwable()));

                return BluetoothProfile.STATE_DISCONNECTED;

                return BluetoothProfile.STATE_DISCONNECTED;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return BluetoothProfile.STATE_DISCONNECTED;

        return BluetoothProfile.STATE_DISCONNECTED;

    }

    }
@@ -236,9 +238,10 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    public BluetoothAvrcpPlayerSettings getPlayerSettings(BluetoothDevice device) {

    public BluetoothAvrcpPlayerSettings getPlayerSettings(BluetoothDevice device) {

        if (DBG) Log.d(TAG, "getPlayerSettings");

        if (DBG) Log.d(TAG, "getPlayerSettings");

        BluetoothAvrcpPlayerSettings settings = null;

        BluetoothAvrcpPlayerSettings settings = null;

        if (mService != null && isEnabled()) {

        final IBluetoothAvrcpController service = mService;

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

            try {

            try {

                settings = mService.getPlayerSettings(device);

                settings = service.getPlayerSettings(device);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Error talking to BT service in getMetadata() " + e);

                Log.e(TAG, "Error talking to BT service in getMetadata() " + e);

                return null;

                return null;
@@ -253,15 +256,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
     */

     */

    public boolean setPlayerApplicationSetting(BluetoothAvrcpPlayerSettings plAppSetting) {

    public boolean setPlayerApplicationSetting(BluetoothAvrcpPlayerSettings plAppSetting) {

        if (DBG) Log.d(TAG, "setPlayerApplicationSetting");

        if (DBG) Log.d(TAG, "setPlayerApplicationSetting");

        if (mService != null && isEnabled()) {

        final IBluetoothAvrcpController service = mService;

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

            try {

            try {

                return mService.setPlayerApplicationSetting(plAppSetting);

                return service.setPlayerApplicationSetting(plAppSetting);

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Error talking to BT service in setPlayerApplicationSetting() " + e);

                Log.e(TAG, "Error talking to BT service in setPlayerApplicationSetting() " + e);

                return false;

                return false;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

        return false;

        return false;

    }

    }
@@ -272,23 +276,23 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    public void sendGroupNavigationCmd(BluetoothDevice device, int keyCode, int keyState) {

    public void sendGroupNavigationCmd(BluetoothDevice device, int keyCode, int keyState) {

        Log.d(TAG, "sendGroupNavigationCmd dev = " + device + " key " + keyCode + " State = "

        Log.d(TAG, "sendGroupNavigationCmd dev = " + device + " key " + keyCode + " State = "

                + keyState);

                + keyState);

        if (mService != null && isEnabled()) {

        final IBluetoothAvrcpController service = mService;

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

            try {

            try {

                mService.sendGroupNavigationCmd(device, keyCode, keyState);

                service.sendGroupNavigationCmd(device, keyCode, keyState);

                return;

                return;

            } catch (RemoteException e) {

            } catch (RemoteException e) {

                Log.e(TAG, "Error talking to BT service in sendGroupNavigationCmd()", e);

                Log.e(TAG, "Error talking to BT service in sendGroupNavigationCmd()", e);

                return;

                return;

            }

            }

        }

        }

        if (mService == null) Log.w(TAG, "Proxy not attached to service");

        if (service == null) Log.w(TAG, "Proxy not attached to service");

    }

    }





    private final ServiceConnection mConnection = new ServiceConnection() {

    private final ServiceConnection mConnection = new ServiceConnection() {

        public void onServiceConnected(ComponentName className, IBinder service) {

        public void onServiceConnected(ComponentName className, IBinder service) {

            if (DBG) Log.d(TAG, "Proxy object connected");

            if (DBG) Log.d(TAG, "Proxy object connected");

            mService = IBluetoothAvrcpController.Stub.asInterface(Binder.allowBlocking(service));

            mService = IBluetoothAvrcpController.Stub.asInterface(Binder.allowBlocking(service));



            if (mServiceListener != null) {

            if (mServiceListener != null) {

                mServiceListener.onServiceConnected(BluetoothProfile.AVRCP_CONTROLLER,

                mServiceListener.onServiceConnected(BluetoothProfile.AVRCP_CONTROLLER,

                        BluetoothAvrcpController.this);

                        BluetoothAvrcpController.this);
@@ -305,15 +309,11 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
    };

    };





    private boolean isEnabled() {

    private boolean isEnabled() {

        if (mAdapter.getState() == BluetoothAdapter.STATE_ON) return true;

        return mAdapter.getState() == BluetoothAdapter.STATE_ON;

        return false;

    }

    }





    private boolean isValidDevice(BluetoothDevice device) {

    private static boolean isValidDevice(BluetoothDevice device) {

        if (device == null) return false;

        return device != null && BluetoothAdapter.checkBluetoothAddress(device.getAddress());



        if (BluetoothAdapter.checkBluetoothAddress(device.getAddress())) return true;

        return false;

    }

    }





    private static void log(String msg) {

    private static void log(String msg) {

core/java/android/bluetooth/BluetoothDevice.java

+86 −57

File changed.

Preview size limit exceeded, changes collapsed.

core/java/android/bluetooth/BluetoothHeadset.java

+102 −79

File changed.

Preview size limit exceeded, changes collapsed.

core/java/android/bluetooth/BluetoothHeadsetClient.java

+105 −83

File changed.

Preview size limit exceeded, changes collapsed.