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

Commit 5d36a177 authored by Jerry Zhang's avatar Jerry Zhang
Browse files

Refactor setCurrentFunction and setUsbDataUnlocked into one method 

This allows us to get rid of an extraneous config switch and
simplify some code.

Test: Manually change usb configuration
Change-Id: Id78da530ff485ecd7a915056832eec1dd8c91954
parent 2b03a7d0

src/com/android/settings/DevelopmentSettings.java

+2 −3
Original line number Diff line number Diff line
@@ -1707,11 +1707,10 @@ public class DevelopmentSettings extends RestrictedSettingsFragment 
    private void writeUsbConfigurationOption(Object newValue) {

        UsbManager manager = (UsbManager)getActivity().getSystemService(Context.USB_SERVICE);

        String function = newValue.toString();

        manager.setCurrentFunction(function);

        if (function.equals("none")) {

            manager.setUsbDataUnlocked(false);

            manager.setCurrentFunction(function, false);

        } else {

            manager.setUsbDataUnlocked(true);

            manager.setCurrentFunction(function, true);

        }

    }

src/com/android/settings/deviceinfo/UsbBackend.java

+4 −8
Original line number Diff line number Diff line
@@ -101,20 +101,16 @@ public class UsbBackend { 
    private void setUsbFunction(int mode) {

        switch (mode) {

            case MODE_DATA_MTP:

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_MTP);

                mUsbManager.setUsbDataUnlocked(true);

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_MTP, true);

                break;

            case MODE_DATA_PTP:

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_PTP);

                mUsbManager.setUsbDataUnlocked(true);

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_PTP, true);

                break;

            case MODE_DATA_MIDI:

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_MIDI);

                mUsbManager.setUsbDataUnlocked(true);

                mUsbManager.setCurrentFunction(UsbManager.USB_FUNCTION_MIDI, true);

                break;

            default:

                mUsbManager.setCurrentFunction(null);

                mUsbManager.setUsbDataUnlocked(false);

                mUsbManager.setCurrentFunction(null, false);

                break;

        }

    }