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

Commit f235ec45 authored by Dave Mankoff's avatar Dave Mankoff
Browse files

Give a new Handler to each injected Executor. 

We still use the same looper (and thread) for the Executors.
This simply ensures that each Executor has its own queue of work.

Bug: 145135056
Test: manual
Change-Id: Id06f59f88047d4616a70d8c3c7fe1a7478cb2c58
parent 9d4f6c0c

packages/SystemUI/src/com/android/systemui/util/concurrency/ConcurrencyModule.java

+13 −12
Original line number Diff line number Diff line
@@ -18,11 +18,12 @@ package com.android.systemui.util.concurrency; 


import android.content.Context;

import android.os.Handler;

import android.os.Looper;



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

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

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

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

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

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



import java.util.concurrent.Executor;
@@ -38,8 +39,8 @@ public abstract class ConcurrencyModule { 
     * Provide a Background-Thread Executor by default.

     */

    @Provides

    public static Executor provideExecutor(@BgHandler Handler handler) {

        return new ExecutorImpl(handler);

    public static Executor provideExecutor(@BgLooper Looper looper) {

        return new ExecutorImpl(new Handler(looper));

    }



    /**
@@ -47,8 +48,8 @@ public abstract class ConcurrencyModule { 
     */

    @Provides

    @Background

    public static Executor provideBackgroundExecutor(@BgHandler Handler handler) {

        return new ExecutorImpl(handler);

    public static Executor provideBackgroundExecutor(@BgLooper Looper looper) {

        return new ExecutorImpl(new Handler(looper));

    }



    /**
@@ -64,8 +65,8 @@ public abstract class ConcurrencyModule { 
     * Provide a Background-Thread Executor by default.

     */

    @Provides

    public static DelayableExecutor provideDelayableExecutor(@BgHandler Handler handler) {

        return new ExecutorImpl(handler);

    public static DelayableExecutor provideDelayableExecutor(@BgLooper Looper looper) {

        return new ExecutorImpl(new Handler(looper));

    }



    /**
@@ -73,8 +74,8 @@ public abstract class ConcurrencyModule { 
     */

    @Provides

    @Background

    public static DelayableExecutor provideBackgroundDelayableExecutor(@BgHandler Handler handler) {

        return new ExecutorImpl(handler);

    public static DelayableExecutor provideBackgroundDelayableExecutor(@BgLooper Looper looper) {

        return new ExecutorImpl(new Handler(looper));

    }



    /**
@@ -82,7 +83,7 @@ public abstract class ConcurrencyModule { 
     */

    @Provides

    @Main

    public static DelayableExecutor provideMainDelayableExecutor(@MainHandler Handler handler) {

        return new ExecutorImpl(handler);

    public static DelayableExecutor provideMainDelayableExecutor(@MainLooper Looper looper) {

        return new ExecutorImpl(new Handler(looper));

    }

}