Merge "[Dagger] Remove Dependency.get from BrightnessController." into sc-v2-dev

This commit is contained in:
Caitlin Cassidy
2021-08-09 17:32:17 +00:00
committed by Android (Google) Code Review
2 changed files with 26 additions and 9 deletions

View File

@@ -30,7 +30,6 @@ import android.hardware.display.DisplayManager.DisplayListener;
import android.net.Uri; import android.net.Uri;
import android.os.AsyncTask; import android.os.AsyncTask;
import android.os.Handler; import android.os.Handler;
import android.os.Looper;
import android.os.Message; import android.os.Message;
import android.os.PowerManager; import android.os.PowerManager;
import android.os.RemoteException; import android.os.RemoteException;
@@ -47,8 +46,8 @@ import com.android.internal.display.BrightnessSynchronizer;
import com.android.internal.logging.MetricsLogger; import com.android.internal.logging.MetricsLogger;
import com.android.internal.logging.nano.MetricsProto.MetricsEvent; import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
import com.android.settingslib.RestrictedLockUtilsInternal; import com.android.settingslib.RestrictedLockUtilsInternal;
import com.android.systemui.Dependency;
import com.android.systemui.broadcast.BroadcastDispatcher; import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.dagger.qualifiers.Background;
import com.android.systemui.settings.CurrentUserTracker; import com.android.systemui.settings.CurrentUserTracker;
import com.android.systemui.statusbar.policy.BrightnessMirrorController; import com.android.systemui.statusbar.policy.BrightnessMirrorController;
@@ -288,12 +287,15 @@ public class BrightnessController implements ToggleSlider.Listener, MirroredBrig
} }
}; };
public BrightnessController(Context context, ToggleSlider control, public BrightnessController(
BroadcastDispatcher broadcastDispatcher) { Context context,
ToggleSlider control,
BroadcastDispatcher broadcastDispatcher,
@Background Handler bgHandler) {
mContext = context; mContext = context;
mControl = control; mControl = control;
mControl.setMax(GAMMA_SPACE_MAX); mControl.setMax(GAMMA_SPACE_MAX);
mBackgroundHandler = new Handler((Looper) Dependency.get(Dependency.BG_LOOPER)); mBackgroundHandler = bgHandler;
mUserTracker = new CurrentUserTracker(broadcastDispatcher) { mUserTracker = new CurrentUserTracker(broadcastDispatcher) {
@Override @Override
public void onUserSwitched(int newUserId) { public void onUserSwitched(int newUserId) {
@@ -464,16 +466,25 @@ public class BrightnessController implements ToggleSlider.Listener, MirroredBrig
public static class Factory { public static class Factory {
private final Context mContext; private final Context mContext;
private final BroadcastDispatcher mBroadcastDispatcher; private final BroadcastDispatcher mBroadcastDispatcher;
private final Handler mBackgroundHandler;
@Inject @Inject
public Factory(Context context, BroadcastDispatcher broadcastDispatcher) { public Factory(
Context context,
BroadcastDispatcher broadcastDispatcher,
@Background Handler bgHandler) {
mContext = context; mContext = context;
mBroadcastDispatcher = broadcastDispatcher; mBroadcastDispatcher = broadcastDispatcher;
mBackgroundHandler = bgHandler;
} }
/** Create a {@link BrightnessController} */ /** Create a {@link BrightnessController} */
public BrightnessController create(ToggleSlider toggleSlider) { public BrightnessController create(ToggleSlider toggleSlider) {
return new BrightnessController(mContext, toggleSlider, mBroadcastDispatcher); return new BrightnessController(
mContext,
toggleSlider,
mBroadcastDispatcher,
mBackgroundHandler);
} }
} }

View File

@@ -21,6 +21,7 @@ import static android.view.ViewGroup.LayoutParams.WRAP_CONTENT;
import android.app.Activity; import android.app.Activity;
import android.os.Bundle; import android.os.Bundle;
import android.os.Handler;
import android.view.Gravity; import android.view.Gravity;
import android.view.KeyEvent; import android.view.KeyEvent;
import android.view.View; import android.view.View;
@@ -32,6 +33,7 @@ import com.android.internal.logging.MetricsLogger;
import com.android.internal.logging.nano.MetricsProto.MetricsEvent; import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
import com.android.systemui.R; import com.android.systemui.R;
import com.android.systemui.broadcast.BroadcastDispatcher; import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.dagger.qualifiers.Background;
import javax.inject.Inject; import javax.inject.Inject;
@@ -41,13 +43,16 @@ public class BrightnessDialog extends Activity {
private BrightnessController mBrightnessController; private BrightnessController mBrightnessController;
private final BrightnessSlider.Factory mToggleSliderFactory; private final BrightnessSlider.Factory mToggleSliderFactory;
private final BroadcastDispatcher mBroadcastDispatcher; private final BroadcastDispatcher mBroadcastDispatcher;
private final Handler mBackgroundHandler;
@Inject @Inject
public BrightnessDialog( public BrightnessDialog(
BroadcastDispatcher broadcastDispatcher, BroadcastDispatcher broadcastDispatcher,
BrightnessSlider.Factory factory) { BrightnessSlider.Factory factory,
@Background Handler bgHandler) {
mBroadcastDispatcher = broadcastDispatcher; mBroadcastDispatcher = broadcastDispatcher;
mToggleSliderFactory = factory; mToggleSliderFactory = factory;
mBackgroundHandler = bgHandler;
} }
@@ -76,7 +81,8 @@ public class BrightnessDialog extends Activity {
controller.init(); controller.init();
frame.addView(controller.getRootView(), MATCH_PARENT, WRAP_CONTENT); frame.addView(controller.getRootView(), MATCH_PARENT, WRAP_CONTENT);
mBrightnessController = new BrightnessController(this, controller, mBroadcastDispatcher); mBrightnessController = new BrightnessController(
this, controller, mBroadcastDispatcher, mBackgroundHandler);
} }
@Override @Override