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

Commit f5bc00c9 authored by Robert Wu's avatar Robert Wu Committed by Automerger Merge Worker
Browse files

Merge "MidiService: Fix USB removeDeviceConnection" into tm-dev am: 5fa99c33 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18169657



Change-Id: I7136d3888530191c7fe7e23eaeb5240331c25dac
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 55843342 5fa99c33

services/midi/java/com/android/server/midi/MidiService.java

+20 −7
Original line number Diff line number Diff line
@@ -238,7 +238,7 @@ public class MidiService extends IMidiManager.Stub { 
            }

        }



        // called from Device.close()

        // called from Device.closeLocked()

        public void removeDeviceConnection(DeviceConnection connection) {

            mDeviceConnections.remove(connection.getToken());

            if (mListeners.size() == 0 && mDeviceConnections.size() == 0) {
@@ -294,12 +294,6 @@ public class MidiService extends IMidiManager.Stub { 
            }



            for (DeviceConnection connection : mDeviceConnections.values()) {

                if (connection.getDevice().getDeviceInfo().getType()

                        == MidiDeviceInfo.TYPE_USB) {

                    synchronized (mUsbMidiLock) {

                        removeUsbMidiDeviceLocked(connection.getDevice().getDeviceInfo());

                    }

                }

                connection.getDevice().removeDeviceConnection(connection);

            }

        }
@@ -541,6 +535,13 @@ public class MidiService extends IMidiManager.Stub { 
                synchronized (mDeviceConnections) {

                    mDeviceConnections.remove(connection);



                    if (connection.getDevice().getDeviceInfo().getType()

                            == MidiDeviceInfo.TYPE_USB) {

                        synchronized (mUsbMidiLock) {

                            removeUsbMidiDeviceLocked(connection.getDevice().getDeviceInfo());

                        }

                    }



                    if (mDeviceConnections.size() == 0 && mServiceConnection != null) {

                        mContext.unbindService(mServiceConnection);

                        mServiceConnection = null;
@@ -559,6 +560,12 @@ public class MidiService extends IMidiManager.Stub { 
        public void closeLocked() {

            synchronized (mDeviceConnections) {

                for (DeviceConnection connection : mDeviceConnections) {

                    if (connection.getDevice().getDeviceInfo().getType()

                            == MidiDeviceInfo.TYPE_USB) {

                        synchronized (mUsbMidiLock) {

                            removeUsbMidiDeviceLocked(connection.getDevice().getDeviceInfo());

                        }

                    }

                    connection.getClient().removeDeviceConnection(connection);

                }

                mDeviceConnections.clear();
@@ -1401,6 +1408,8 @@ public class MidiService extends IMidiManager.Stub { 
        String deviceName = extractUsbDeviceName(name);

        String tagName = extractUsbDeviceTag(name);



        Log.i(TAG, "Checking " + deviceName + " " + tagName);



        // Only one MIDI 2.0 device can be used at once.

        // Multiple MIDI 1.0 devices can be used at once.

        if (mUsbMidiUniversalDeviceInUse.contains(deviceName)
@@ -1420,6 +1429,8 @@ public class MidiService extends IMidiManager.Stub { 
        String deviceName = extractUsbDeviceName(name);

        String tagName = extractUsbDeviceTag(name);



        Log.i(TAG, "Adding " + deviceName + " " + tagName);



        if ((tagName).equals(MIDI_UNIVERSAL_STRING)) {

            mUsbMidiUniversalDeviceInUse.add(deviceName);

        } else if ((tagName).equals(MIDI_LEGACY_STRING)) {
@@ -1437,6 +1448,8 @@ public class MidiService extends IMidiManager.Stub { 
        String deviceName = extractUsbDeviceName(name);

        String tagName = extractUsbDeviceTag(name);



        Log.i(TAG, "Removing " + deviceName + " " + tagName);



        if ((tagName).equals(MIDI_UNIVERSAL_STRING)) {

            mUsbMidiUniversalDeviceInUse.remove(deviceName);

        } else if ((tagName).equals(MIDI_LEGACY_STRING)) {