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

Commit f6b4704c authored by mattsziklay's avatar mattsziklay
Browse files

Set density to DENSITY_DPI_UNDEFINED for display connection events. 

Rather than use the display's density, set density to undefined for disconnect/reconnect events. This will ensure app content resizes properly without locking density from being changed elsewhere.

Bug: 441726870
Test: Manual
Flag: com.android.window.flags.enable_display_disconnect_interaction
Flag: com.android.window.flags.enable_display_reconnect_interaction
Change-Id: Ia101ab40fd86939a343c0300ac1df47ed37b9ebe
parent bb8c568f

libs/WindowManager/Shell/src/com/android/wm/shell/desktopmode/DesktopTasksController.kt

+1 −5
Original line number Diff line number Diff line
@@ -36,6 +36,7 @@ import android.app.WindowConfiguration.WindowingMode 
import android.content.Context

import android.content.Intent

import android.content.pm.PackageManager

import android.content.res.Configuration

import android.graphics.Point

import android.graphics.PointF

import android.graphics.Rect
@@ -843,7 +844,6 @@ class DesktopTasksController( 
                    deskIds,

                    disconnectedDisplayId,

                    destinationDisplayId,

                    destDisplayLayout,

                    userId,

                )

            }
@@ -956,7 +956,6 @@ class DesktopTasksController( 
        deskIds: List<Int>,

        disconnectedDisplayId: Int,

        destinationDisplayId: Int,

        destDisplayLayout: DisplayLayout?,

        userId: Int,

    ) {

        logD("handleProjectedModeDisconnect: moving tasks to non-desktop display")
@@ -1018,7 +1017,6 @@ class DesktopTasksController( 
        val activeDeskId = repository.getPreservedActiveDesk(uniqueDisplayId)

        val wct = WindowContainerTransaction()

        var runOnTransitStartList = mutableListOf<RunOnTransitStart>()

        val destDisplayLayout = displayController.getDisplayLayout(displayId) ?: return

        val tilingReconnectHandler =

            TilingDisplayReconnectEventHandler(repository, snapEventHandler, transitions, displayId)

        val excludedTasks =
@@ -1054,7 +1052,6 @@ class DesktopTasksController( 
                    if (!excludedTasks.contains(taskId)) {

                        addRestoreTaskToDeskChanges(

                                wct = wct,

                                destinationDisplayLayout = destDisplayLayout,

                                deskId = newDeskId,

                                taskId = taskId,

                                userId = userId,
@@ -1091,7 +1088,6 @@ class DesktopTasksController( 


    private fun addRestoreTaskToDeskChanges(

        wct: WindowContainerTransaction,

        destinationDisplayLayout: DisplayLayout,

        deskId: Int,

        taskId: Int,

        userId: Int,

libs/WindowManager/Shell/src/com/android/wm/shell/desktopmode/multidesks/RootTaskDesksOrganizer.kt

+3 −0
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ import android.app.WindowConfiguration.ACTIVITY_TYPE_UNDEFINED 
import android.app.WindowConfiguration.WINDOWING_MODE_FREEFORM

import android.app.WindowConfiguration.WINDOWING_MODE_UNDEFINED

import android.app.WindowConfiguration.windowingModeToString

import android.content.res.Configuration

import android.os.Trace

import android.util.SparseArray

import android.view.SurfaceControl
@@ -213,6 +214,8 @@ class RootTaskDesksOrganizer( 
        // to prevent this when the changes merge.

        wct.setWindowingMode(root.token, WINDOWING_MODE_FREEFORM)

        wct.setWindowingMode(minimizationRoot.token, WINDOWING_MODE_FREEFORM)

        wct.setDensityDpi(root.token, Configuration.DENSITY_DPI_UNDEFINED)

        wct.setDensityDpi(minimizationRoot.token, Configuration.DENSITY_DPI_UNDEFINED)

    }



    override fun activateDesk(wct: WindowContainerTransaction, deskId: Int, skipReorder: Boolean) {