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

Commit 047f0645 authored by Anton Potapov's avatar Anton Potapov
Browse files

Revert "Move NetworkControllerImpl to @LongRunning thread to prevent @Background from being stuck" 

This reverts commit 4db94a0a.

Reason for revert: Performance regression found: b/261773357. I'll rework this with a completely separated thread.

Change-Id: I1ea65fdeeeb1693ba283f446dee9ba92d148442d
Fixes: 261773357
parent 2016b1dc

packages/SystemUI/src/com/android/systemui/statusbar/connectivity/NetworkControllerImpl.java

+3 −6
Original line number Diff line number Diff line
@@ -68,7 +68,7 @@ import com.android.systemui.Dumpable; 
import com.android.systemui.R;

import com.android.systemui.broadcast.BroadcastDispatcher;

import com.android.systemui.dagger.SysUISingleton;

import com.android.systemui.dagger.qualifiers.LongRunning;

import com.android.systemui.dagger.qualifiers.Background;

import com.android.systemui.dagger.qualifiers.Main;

import com.android.systemui.demomode.DemoMode;

import com.android.systemui.demomode.DemoModeController;
@@ -226,15 +226,12 @@ public class NetworkControllerImpl extends BroadcastReceiver 


    /**

     * Construct this controller object and register for updates.

     *

     * {@code @LongRunning} looper and bgExecutor instead {@code @Background} ones are used to

     * address the b/246456655. This can be reverted after b/240663726 is fixed.

     */

    @Inject

    public NetworkControllerImpl(

            Context context,

            @LongRunning Looper bgLooper,

            @LongRunning Executor bgExecutor,

            @Background Looper bgLooper,

            @Background Executor bgExecutor,

            SubscriptionManager subscriptionManager,

            CallbackHandler callbackHandler,

            DeviceProvisionedController deviceProvisionedController,