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

Commit 7f9f8642 authored by William Escande's avatar William Escande
Browse files

SystemServer: use handler for BroadcastReceiver 

Bug: 333764513
Test: m .
Change-Id: I7ee583bcf489eec26f1ee19629300317953d511b
parent ce2f1cad

service/src/com/android/server/bluetooth/BluetoothManagerService.java

+2 −2
Original line number Diff line number Diff line
@@ -659,7 +659,7 @@ class BluetoothManagerService { 
        filter.addAction(Intent.ACTION_SETTING_RESTORED);

        filter.addAction(Intent.ACTION_SHUTDOWN);

        filter.setPriority(IntentFilter.SYSTEM_HIGH_PRIORITY);

        mContext.registerReceiver(mReceiver, filter);

        mContext.registerReceiver(mReceiver, filter, null, mHandler);



        IntentFilter filterUser = new IntentFilter();

        filterUser.addAction(UserManager.ACTION_USER_RESTRICTIONS_CHANGED);
@@ -688,7 +688,7 @@ class BluetoothManagerService { 
                },

                filterUser,

                null,

                null);

                mHandler);



        loadStoredNameAndAddress();

        if (isBluetoothPersistedStateOn()) {

service/tests/src/com/android/server/bluetooth/BluetoothManagerServiceTest.java

+4 −7
Original line number Diff line number Diff line
@@ -95,13 +95,12 @@ public class BluetoothManagerServiceTest { 


    TestLooper mLooper;



    boolean mHasException = false;



    static {

        // Required for reading DeviceConfig.

        InstrumentationRegistry.getInstrumentation()

                .getUiAutomation()

                .adoptShellPermissionIdentity(android.Manifest.permission.READ_DEVICE_CONFIG);

                .adoptShellPermissionIdentity(

                        android.Manifest.permission.INTERACT_ACROSS_USERS_FULL);

    }



    @Before
@@ -168,10 +167,8 @@ public class BluetoothManagerServiceTest { 
            mManagerService = null;

        }

        mLooper.moveTimeForward(120_000); // 120 seconds

        // Do not try to assert if `syncHandler()` already raised an exception for it

        if (!mHasException) {



        assertThat(mLooper.nextMessage()).isNull();

        }

        validateMockitoUsage();

    }