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

Commit a4d446fd authored by Winson Chung's avatar Winson Chung Committed by Android (Google) Code Review
Browse files

Merge "Minor refactoring to support moving code to shell"

parents 584d30b5 95c9fcac
Loading
Loading
Loading
Loading
+3 −3
Original line number Original line Diff line number Diff line
@@ -124,8 +124,8 @@ import com.android.systemui.util.leak.GarbageMonitor;
import com.android.systemui.util.leak.LeakDetector;
import com.android.systemui.util.leak.LeakDetector;
import com.android.systemui.util.leak.LeakReporter;
import com.android.systemui.util.leak.LeakReporter;
import com.android.systemui.util.sensors.AsyncSensorManager;
import com.android.systemui.util.sensors.AsyncSensorManager;
import com.android.systemui.wm.DisplayController;
import com.android.systemui.wm.DisplayImeController;
import com.android.systemui.wm.DisplayImeController;
import com.android.systemui.wm.DisplayWindowController;
import com.android.systemui.wm.SystemWindows;
import com.android.systemui.wm.SystemWindows;


import java.io.FileDescriptor;
import java.io.FileDescriptor;
@@ -324,7 +324,7 @@ public class Dependency {
    @Inject Lazy<CommandQueue> mCommandQueue;
    @Inject Lazy<CommandQueue> mCommandQueue;
    @Inject Lazy<Recents> mRecents;
    @Inject Lazy<Recents> mRecents;
    @Inject Lazy<StatusBar> mStatusBar;
    @Inject Lazy<StatusBar> mStatusBar;
    @Inject Lazy<DisplayWindowController> mDisplayWindowController;
    @Inject Lazy<DisplayController> mDisplayController;
    @Inject Lazy<SystemWindows> mSystemWindows;
    @Inject Lazy<SystemWindows> mSystemWindows;
    @Inject Lazy<DisplayImeController> mDisplayImeController;
    @Inject Lazy<DisplayImeController> mDisplayImeController;
    @Inject Lazy<RecordingController> mRecordingController;
    @Inject Lazy<RecordingController> mRecordingController;
@@ -518,7 +518,7 @@ public class Dependency {
        mProviders.put(CommandQueue.class, mCommandQueue::get);
        mProviders.put(CommandQueue.class, mCommandQueue::get);
        mProviders.put(Recents.class, mRecents::get);
        mProviders.put(Recents.class, mRecents::get);
        mProviders.put(StatusBar.class, mStatusBar::get);
        mProviders.put(StatusBar.class, mStatusBar::get);
        mProviders.put(DisplayWindowController.class, mDisplayWindowController::get);
        mProviders.put(DisplayController.class, mDisplayController::get);
        mProviders.put(SystemWindows.class, mSystemWindows::get);
        mProviders.put(SystemWindows.class, mSystemWindows::get);
        mProviders.put(DisplayImeController.class, mDisplayImeController::get);
        mProviders.put(DisplayImeController.class, mDisplayImeController::get);
        mProviders.put(ProtoTracer.class, mProtoTracer::get);
        mProviders.put(ProtoTracer.class, mProtoTracer::get);
+2 −2
Original line number Original line Diff line number Diff line
@@ -20,13 +20,13 @@ import android.content.Context;
import android.content.res.Configuration;
import android.content.res.Configuration;


import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.wm.DisplayWindowController;
import com.android.systemui.wm.DisplayController;


import java.io.PrintWriter;
import java.io.PrintWriter;


public interface BasePipManager {
public interface BasePipManager {
    void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
    void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
            DisplayWindowController displayWindowController);
            DisplayController displayController);
    void showPictureInPictureMenu();
    void showPictureInPictureMenu();
    default void expandPip() {}
    default void expandPip() {}
    default void hidePipMenu(Runnable onStartCallback, Runnable onEndCallback) {}
    default void hidePipMenu(Runnable onStartCallback, Runnable onEndCallback) {}
+5 −5
Original line number Original line Diff line number Diff line
@@ -28,7 +28,7 @@ import android.os.UserManager;
import com.android.systemui.SystemUI;
import com.android.systemui.SystemUI;
import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.broadcast.BroadcastDispatcher;
import com.android.systemui.statusbar.CommandQueue;
import com.android.systemui.statusbar.CommandQueue;
import com.android.systemui.wm.DisplayWindowController;
import com.android.systemui.wm.DisplayController;


import java.io.FileDescriptor;
import java.io.FileDescriptor;
import java.io.PrintWriter;
import java.io.PrintWriter;
@@ -45,17 +45,17 @@ public class PipUI extends SystemUI implements CommandQueue.Callbacks {
    private final CommandQueue mCommandQueue;
    private final CommandQueue mCommandQueue;
    private BasePipManager mPipManager;
    private BasePipManager mPipManager;
    private final BroadcastDispatcher mBroadcastDispatcher;
    private final BroadcastDispatcher mBroadcastDispatcher;
    private final DisplayWindowController mDisplayWindowController;
    private final DisplayController mDisplayController;
    private boolean mSupportsPip;
    private boolean mSupportsPip;


    @Inject
    @Inject
    public PipUI(Context context, CommandQueue commandQueue,
    public PipUI(Context context, CommandQueue commandQueue,
            BroadcastDispatcher broadcastDispatcher,
            BroadcastDispatcher broadcastDispatcher,
            DisplayWindowController displayWindowController) {
            DisplayController displayController) {
        super(context);
        super(context);
        mBroadcastDispatcher = broadcastDispatcher;
        mBroadcastDispatcher = broadcastDispatcher;
        mCommandQueue = commandQueue;
        mCommandQueue = commandQueue;
        mDisplayWindowController = displayWindowController;
        mDisplayController = displayController;
    }
    }


    @Override
    @Override
@@ -75,7 +75,7 @@ public class PipUI extends SystemUI implements CommandQueue.Callbacks {
        mPipManager = pm.hasSystemFeature(FEATURE_LEANBACK_ONLY)
        mPipManager = pm.hasSystemFeature(FEATURE_LEANBACK_ONLY)
                ? com.android.systemui.pip.tv.PipManager.getInstance()
                ? com.android.systemui.pip.tv.PipManager.getInstance()
                : com.android.systemui.pip.phone.PipManager.getInstance();
                : com.android.systemui.pip.phone.PipManager.getInstance();
        mPipManager.initialize(mContext, mBroadcastDispatcher, mDisplayWindowController);
        mPipManager.initialize(mContext, mBroadcastDispatcher, mDisplayController);


        mCommandQueue.addCallback(this);
        mCommandQueue.addCallback(this);
    }
    }
+5 −4
Original line number Original line Diff line number Diff line
@@ -46,7 +46,8 @@ import com.android.systemui.shared.system.InputConsumerController;
import com.android.systemui.shared.system.PinnedStackListenerForwarder.PinnedStackListener;
import com.android.systemui.shared.system.PinnedStackListenerForwarder.PinnedStackListener;
import com.android.systemui.shared.system.TaskStackChangeListener;
import com.android.systemui.shared.system.TaskStackChangeListener;
import com.android.systemui.shared.system.WindowManagerWrapper;
import com.android.systemui.shared.system.WindowManagerWrapper;
import com.android.systemui.wm.DisplayWindowController;
import com.android.systemui.wm.DisplayChangeController;
import com.android.systemui.wm.DisplayController;


import java.io.PrintWriter;
import java.io.PrintWriter;


@@ -79,7 +80,7 @@ public class PipManager implements BasePipManager {
    /**
    /**
     * Handler for display rotation changes.
     * Handler for display rotation changes.
     */
     */
    private final DisplayWindowController.OnDisplayWindowRotationController mRotationController = (
    private final DisplayChangeController.OnDisplayChangingListener mRotationController = (
            int displayId, int fromRotation, int toRotation, WindowContainerTransaction t) -> {
            int displayId, int fromRotation, int toRotation, WindowContainerTransaction t) -> {
        final boolean changed = mPipBoundsHandler.onDisplayRotationChanged(mTmpNormalBounds,
        final boolean changed = mPipBoundsHandler.onDisplayRotationChanged(mTmpNormalBounds,
                displayId, fromRotation, toRotation, t);
                displayId, fromRotation, toRotation, t);
@@ -230,7 +231,7 @@ public class PipManager implements BasePipManager {
     * Initializes {@link PipManager}.
     * Initializes {@link PipManager}.
     */
     */
    public void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
    public void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
            DisplayWindowController displayWindowController) {
            DisplayController displayController) {
        mContext = context;
        mContext = context;
        mActivityManager = ActivityManager.getService();
        mActivityManager = ActivityManager.getService();
        mActivityTaskManager = ActivityTaskManager.getService();
        mActivityTaskManager = ActivityTaskManager.getService();
@@ -251,7 +252,7 @@ public class PipManager implements BasePipManager {
                mMenuController, mInputConsumerController, mPipBoundsHandler);
                mMenuController, mInputConsumerController, mPipBoundsHandler);
        mAppOpsListener = new PipAppOpsListener(context, mActivityManager,
        mAppOpsListener = new PipAppOpsListener(context, mActivityManager,
                mTouchHandler.getMotionHelper());
                mTouchHandler.getMotionHelper());
        displayWindowController.addRotationController(mRotationController);
        displayController.addDisplayChangingController(mRotationController);


        // If SystemUI restart, and it already existed a pinned stack,
        // If SystemUI restart, and it already existed a pinned stack,
        // register the pip input consumer to ensure touch can send to it.
        // register the pip input consumer to ensure touch can send to it.
+2 −2
Original line number Original line Diff line number Diff line
@@ -55,7 +55,7 @@ import com.android.systemui.shared.system.ActivityManagerWrapper;
import com.android.systemui.shared.system.PinnedStackListenerForwarder.PinnedStackListener;
import com.android.systemui.shared.system.PinnedStackListenerForwarder.PinnedStackListener;
import com.android.systemui.shared.system.TaskStackChangeListener;
import com.android.systemui.shared.system.TaskStackChangeListener;
import com.android.systemui.shared.system.WindowManagerWrapper;
import com.android.systemui.shared.system.WindowManagerWrapper;
import com.android.systemui.wm.DisplayWindowController;
import com.android.systemui.wm.DisplayController;


import java.util.ArrayList;
import java.util.ArrayList;
import java.util.List;
import java.util.List;
@@ -230,7 +230,7 @@ public class PipManager implements BasePipManager {
     * Initializes {@link PipManager}.
     * Initializes {@link PipManager}.
     */
     */
    public void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
    public void initialize(Context context, BroadcastDispatcher broadcastDispatcher,
            DisplayWindowController displayWindowController) {
            DisplayController displayController) {
        if (mInitialized) {
        if (mInitialized) {
            return;
            return;
        }
        }
Loading