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

Commit 3b860c3c authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Revert "Add a config to turn on window drag"" into main

parents 51dac5f7 4caaf42b
Loading
Loading
Loading
Loading
+0 −3
Original line number Diff line number Diff line
@@ -188,7 +188,4 @@
    <string-array name="desktop_windowing_app_handle_education_allowlist_apps"></string-array>
    <!-- Apps that can trigger Desktop Windowing App-To-Web Education -->
    <string-array name="desktop_windowing_app_to_web_education_allowlist_apps"></string-array>

    <!-- Whether this device allows to drag windows across displays. -->
    <bool name="config_enableConnectedDisplayWindowDrag">true</bool>
</resources>
+1 −5
Original line number Diff line number Diff line
@@ -37,7 +37,6 @@ import android.window.WindowContainerTransaction;
import androidx.annotation.BinderThread;

import com.android.window.flags.Flags;
import com.android.wm.shell.R;
import com.android.wm.shell.common.DisplayChangeController.OnDisplayChangingListener;
import com.android.wm.shell.shared.annotations.ShellMainThread;
import com.android.wm.shell.sysui.ShellInit;
@@ -92,10 +91,7 @@ public class DisplayController {
                onDisplayAdded(displayIds[i]);
            }

            final boolean enableConnectedDisplayWindowDrag =
                    mContext.getResources()
                            .getBoolean(R.bool.config_enableConnectedDisplayWindowDrag);
            if (Flags.enableConnectedDisplaysWindowDrag() && enableConnectedDisplayWindowDrag) {
            if (Flags.enableConnectedDisplaysWindowDrag()) {
                mDisplayManager.registerTopologyListener(mMainExecutor,
                        this::onDisplayTopologyChanged);
                onDisplayTopologyChanged(mDisplayManager.getDisplayTopology());
+4 −21
Original line number Diff line number Diff line
@@ -29,7 +29,6 @@ import android.window.TransitionRequestInfo
import android.window.WindowContainerTransaction
import com.android.internal.jank.Cuj
import com.android.internal.jank.InteractionJankMonitor
import com.android.wm.shell.R
import com.android.wm.shell.ShellTaskOrganizer
import com.android.wm.shell.common.DisplayController
import com.android.wm.shell.common.MultiDisplayDragMoveBoundsCalculator
@@ -68,8 +67,6 @@ class MultiDisplayVeiledResizeTaskPositioner(
                (ctrlType and DragPositioningCallback.CTRL_TYPE_LEFT) != 0 ||
                (ctrlType and DragPositioningCallback.CTRL_TYPE_RIGHT) != 0

    private val enableMultiDisplayWindowDrag: Boolean

    @DragPositioningCallback.CtrlType private var ctrlType = 0
    private var isResizingOrAnimatingResize = false
    @Surface.Rotation private var rotation = 0
@@ -96,10 +93,6 @@ class MultiDisplayVeiledResizeTaskPositioner(

    init {
        dragEventListeners.add(dragEventListener)
        enableMultiDisplayWindowDrag =
            desktopWindowDecoration.mDecorWindowContext.resources.getBoolean(
                R.bool.config_enableConnectedDisplayWindowDrag
            )
    }

    override fun onDragPositioningStart(ctrlType: Int, displayId: Int, x: Float, y: Float): Rect {
@@ -175,13 +168,8 @@ class MultiDisplayVeiledResizeTaskPositioner(
            val startDisplayLayout = displayController.getDisplayLayout(startDisplayId)
            val currentDisplayLayout = displayController.getDisplayLayout(displayId)

            if (
                startDisplayLayout == null ||
                    currentDisplayLayout == null ||
                    !enableMultiDisplayWindowDrag
            ) {
                // Fall back to single-display drag behavior if any display layout is unavailable or
                // it's explicitly disabled.
            if (startDisplayLayout == null || currentDisplayLayout == null) {
                // Fall back to single-display drag behavior if any display layout is unavailable.
                DragPositioningCallbackUtility.setPositionOnDrag(
                    desktopWindowDecoration,
                    repositionTaskBounds,
@@ -249,13 +237,8 @@ class MultiDisplayVeiledResizeTaskPositioner(
            val startDisplayLayout = displayController.getDisplayLayout(startDisplayId)
            val currentDisplayLayout = displayController.getDisplayLayout(displayId)

            if (
                startDisplayLayout == null ||
                    currentDisplayLayout == null ||
                    !enableMultiDisplayWindowDrag
            ) {
                // Fall back to single-display drag behavior if any display layout is unavailable or
                // it's explicitly disabled.
            if (startDisplayLayout == null || currentDisplayLayout == null) {
                // Fall back to single-display drag behavior if any display layout is unavailable.
                DragPositioningCallbackUtility.updateTaskBounds(
                    repositionTaskBounds,
                    taskBoundsAtDragStart,