RESTRICT AUTOMERGE Ensure that DozeFactory uses Main Thread Handler.

Fixes: 157220475
Test: atest SystemUITests
Change-Id: I57a9c67aa4d2bcb54a31128f31224ff01935058f
This commit is contained in:
Dave Mankoff
2020-05-26 15:05:07 -04:00
parent 175d91f6ee
commit 10d4b8de11
3 changed files with 11 additions and 7 deletions

View File

@@ -96,12 +96,6 @@ public class DependencyProvider {
return new AmbientDisplayConfiguration(context);
}
/** */
@Provides
public Handler provideHandler() {
return new Handler();
}
@Singleton
@Provides
public DataSaverController provideDataSaverController(NetworkController networkController) {

View File

@@ -28,6 +28,7 @@ import android.os.Handler;
import com.android.keyguard.KeyguardUpdateMonitor;
import com.android.systemui.R;
import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.dagger.qualifiers.Main;
import com.android.systemui.dock.DockManager;
import com.android.systemui.keyguard.WakefulnessLifecycle;
import com.android.systemui.plugins.FalsingManager;
@@ -69,7 +70,7 @@ public class DozeFactory {
WakefulnessLifecycle wakefulnessLifecycle, KeyguardUpdateMonitor keyguardUpdateMonitor,
DockManager dockManager, @Nullable IWallpaperManager wallpaperManager,
ProximitySensor proximitySensor,
DelayedWakeLock.Builder delayedWakeLockBuilder, Handler handler,
DelayedWakeLock.Builder delayedWakeLockBuilder, @Main Handler handler,
DelayableExecutor delayableExecutor,
BiometricUnlockController biometricUnlockController,
BroadcastDispatcher broadcastDispatcher, DozeHost dozeHost) {

View File

@@ -91,6 +91,15 @@ public abstract class ConcurrencyModule {
return new Handler(mainLooper);
}
/**
* @deprecated Please specify @Main or @Background when injecting a Handler or use an Executor.
*/
@Deprecated
@Provides
public static Handler provideHandler() {
return new Handler();
}
/**
* Provide a Background-Thread Executor by default.
*/