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

Commit b913a1e6 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Bluetooth: Thread-safe binder invocation" into oc-mr1-dev

parents f6b8e4fd 0146463d

core/java/android/bluetooth/BluetoothA2dpSink.java

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


    private Context mContext;

    private ServiceListener mServiceListener;

    private IBluetoothA2dpSink mService;

    private volatile IBluetoothA2dpSink mService;

    private BluetoothAdapter mAdapter;



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

    public boolean connect(BluetoothDevice device) {

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

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

            isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.connect(device);

                return service.connect(device);

            } catch (RemoteException e) {

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

                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;

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

    public boolean disconnect(BluetoothDevice device) {

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

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

            isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.disconnect(device);

                return service.disconnect(device);

            } catch (RemoteException e) {

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

                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;

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

    public List<BluetoothDevice> getConnectedDevices() {

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

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

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.getConnectedDevices();

                return service.getConnectedDevices();

            } catch (RemoteException e) {

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

                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>();

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

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

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

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

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.getDevicesMatchingConnectionStates(states);

                return service.getDevicesMatchingConnectionStates(states);

            } catch (RemoteException e) {

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

                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>();

    }
@@ -332,16 +334,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

    public int getConnectionState(BluetoothDevice device) {

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

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.getConnectionState(device);

                return service.getConnectionState(device);

            } catch (RemoteException e) {

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

                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;

    }
@@ -358,16 +360,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

          public BluetoothAudioConfig getAudioConfig(BluetoothDevice device) {

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

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.getAudioConfig(device);

                return service.getAudioConfig(device);

            } catch (RemoteException e) {

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

                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;

    }
@@ -388,20 +390,20 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

    public boolean setPriority(BluetoothDevice device, int priority) {

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

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

            if (priority != BluetoothProfile.PRIORITY_OFF &&

                priority != BluetoothProfile.PRIORITY_ON){

        final IBluetoothA2dpSink service = mService;

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

            if (priority != BluetoothProfile.PRIORITY_OFF

                    && priority != BluetoothProfile.PRIORITY_ON) {

                return false;

            }

            try {

                return mService.setPriority(device, priority);

                return service.setPriority(device, priority);

            } catch (RemoteException e) {

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

                   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;

    }
@@ -420,16 +422,16 @@ public final class BluetoothA2dpSink implements BluetoothProfile { 
     */

    public int getPriority(BluetoothDevice device) {

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

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.getPriority(device);

                return service.getPriority(device);

            } catch (RemoteException e) {

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

                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;

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

     */

    public boolean isA2dpPlaying(BluetoothDevice device) {

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

        final IBluetoothA2dpSink service = mService;

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

            try {

                return mService.isA2dpPlaying(device);

                return service.isA2dpPlaying(device);

            } catch (RemoteException e) {

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

                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;

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

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

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



            if (mServiceListener != null) {

                mServiceListener.onServiceConnected(BluetoothProfile.A2DP_SINK,

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



    private boolean isEnabled() {

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

       return false;

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

    }



    private boolean isValidDevice(BluetoothDevice device) {

       if (device == null) return false;



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

       return false;

    private static boolean isValidDevice(BluetoothDevice device) {

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

    }



    private static void log(String msg) {

core/java/android/bluetooth/BluetoothAvrcpController.java

+29 −30
Original line number Diff line number Diff line
@@ -20,8 +20,6 @@ import android.content.ComponentName; 
import android.content.Context;

import android.content.Intent;

import android.content.ServiceConnection;

import android.media.MediaMetadata;

import android.media.session.PlaybackState;

import android.os.Binder;

import android.os.IBinder;

import android.os.RemoteException;
@@ -83,7 +81,7 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 


    private Context mContext;

    private ServiceListener mServiceListener;

    private IBluetoothAvrcpController mService;

    private volatile IBluetoothAvrcpController mService;

    private BluetoothAdapter mAdapter;



    final private IBluetoothStateChangeCallback mBluetoothStateChangeCallback =
@@ -180,15 +178,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
     */

    public List<BluetoothDevice> getConnectedDevices() {

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

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

        final IBluetoothAvrcpController service = mService;

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

            try {

                return mService.getConnectedDevices();

                return service.getConnectedDevices();

            } catch (RemoteException e) {

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

                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>();

    }
@@ -197,15 +196,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
     */

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

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

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

        final IBluetoothAvrcpController service = mService;

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

            try {

                return mService.getDevicesMatchingConnectionStates(states);

                return service.getDevicesMatchingConnectionStates(states);

            } catch (RemoteException e) {

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

                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>();

    }
@@ -214,16 +214,16 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
     */

    public int getConnectionState(BluetoothDevice device) {

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

        if (mService != null && isEnabled()

            && isValidDevice(device)) {

        final IBluetoothAvrcpController service = mService;

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

            try {

                return mService.getConnectionState(device);

                return service.getConnectionState(device);

            } catch (RemoteException e) {

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

                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;

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

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

        BluetoothAvrcpPlayerSettings settings = null;

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

        final IBluetoothAvrcpController service = mService;

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

            try {

                settings = mService.getPlayerSettings(device);

                settings = service.getPlayerSettings(device);

            } catch (RemoteException e) {

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

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

    public boolean setPlayerApplicationSetting(BluetoothAvrcpPlayerSettings plAppSetting) {

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

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

        final IBluetoothAvrcpController service = mService;

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

            try {

                return mService.setPlayerApplicationSetting(plAppSetting);

                return service.setPlayerApplicationSetting(plAppSetting);

            } catch (RemoteException e) {

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

                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;

    }
@@ -269,24 +271,25 @@ public final class BluetoothAvrcpController implements BluetoothProfile { 
     * possible keycode values: next_grp, previous_grp defined above

     */

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

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

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

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

                + keyState);

        final IBluetoothAvrcpController service = mService;

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

            try {

                mService.sendGroupNavigationCmd(device, keyCode, keyState);

                service.sendGroupNavigationCmd(device, keyCode, keyState);

                return;

            } catch (RemoteException e) {

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

                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() {

        public void onServiceConnected(ComponentName className, IBinder service) {

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

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



            if (mServiceListener != null) {

                mServiceListener.onServiceConnected(BluetoothProfile.AVRCP_CONTROLLER,

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



    private boolean isEnabled() {

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

       return false;

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

    }



    private boolean isValidDevice(BluetoothDevice device) {

       if (device == null) return false;



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

       return false;

    private static boolean isValidDevice(BluetoothDevice device) {

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

    }



    private static void log(String msg) {

core/java/android/bluetooth/BluetoothDevice.java

+198 −136

File changed.

Preview size limit exceeded, changes collapsed.

core/java/android/bluetooth/BluetoothHeadset.java

+125 −102

File changed.

Preview size limit exceeded, changes collapsed.

core/java/android/bluetooth/BluetoothHeadsetClient.java

+118 −109

File changed.

Preview size limit exceeded, changes collapsed.