summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Robin Lee <rgl@google.com> 2025-01-24 04:54:04 -0800
committer Android (Google) Code Review <android-gerrit@google.com> 2025-01-24 04:54:09 -0800
commit4caaf42b8d080c7b026ea3ba64ee23f702abb243 (patch)
tree010f3c7e5d18d34e1b3f4229d4f86043807263c2
parent9a0b58ce46a7d9ed4255d5a396369b9b34e5399c (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
-rw-r--r--libs/WindowManager/Shell/res/values/config.xml3
-rw-r--r--libs/WindowManager/Shell/src/com/android/wm/shell/common/DisplayController.java6
-rw-r--r--libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/MultiDisplayVeiledResizeTaskPositioner.kt25
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,