diff options
author | 2025-01-24 04:54:04 -0800 | |
---|---|---|
committer | 2025-01-24 04:54:09 -0800 | |
commit | 4caaf42b8d080c7b026ea3ba64ee23f702abb243 (patch) | |
tree | 010f3c7e5d18d34e1b3f4229d4f86043807263c2 | |
parent | 9a0b58ce46a7d9ed4255d5a396369b9b34e5399c (diff) |
Revert "Add a config to turn on window drag"
Revert submission 31309007-config_enableConnectedDisplaysWindowDrag
Reason for revert: CTS failures to to SystemUI crash on CF
Bug: 391097302
Bug: 391992975
Bug: 392053843
Test: atest CtsWindowManagerDeviceAnimations
Test: atest CtsWindowManagerBackgroundActivityTestCases
Reverted changes: /q/submissionid:31309007-config_enableConnectedDisplaysWindowDrag
Change-Id: I15d86498df1a41ab3273b4060ffb4930dd502fc3
3 files changed, 5 insertions, 29 deletions
diff --git a/libs/WindowManager/Shell/res/values/config.xml b/libs/WindowManager/Shell/res/values/config.xml index c38d459f64d3..59e6c7786cb8 100644 --- a/libs/WindowManager/Shell/res/values/config.xml +++ b/libs/WindowManager/Shell/res/values/config.xml @@ -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> diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/common/DisplayController.java b/libs/WindowManager/Shell/src/com/android/wm/shell/common/DisplayController.java index 71f30f39136a..72be066fc7a7 100644 --- a/libs/WindowManager/Shell/src/com/android/wm/shell/common/DisplayController.java +++ b/libs/WindowManager/Shell/src/com/android/wm/shell/common/DisplayController.java @@ -36,7 +36,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; @@ -88,10 +87,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()); diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/MultiDisplayVeiledResizeTaskPositioner.kt b/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/MultiDisplayVeiledResizeTaskPositioner.kt index 33f0432372d0..07496eb0e526 100644 --- a/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/MultiDisplayVeiledResizeTaskPositioner.kt +++ b/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/MultiDisplayVeiledResizeTaskPositioner.kt @@ -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, |