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

Commit b71bcc48 authored by Andrii Kulian's avatar Andrii Kulian Committed by android-build-merger
Browse files

Merge "Set parent for WindowProcessController" into qt-dev 

am: b124e0a4

Change-Id: Ibf69628e7de15cb0ff10a5aefe341b8c7bdb58a3
parents d605ac12 b124e0a4

services/core/java/com/android/server/wm/WindowProcessController.java

+4 −1
Original line number Diff line number Diff line
@@ -439,7 +439,10 @@ public class WindowProcessController extends ConfigurationContainer<Configuratio 


    @Override

    protected ConfigurationContainer getParent() {

        return null;

        // Returning RootActivityContainer as the parent, so that this process controller always

        // has full configuration and overrides (e.g. from display) are always added on top of

        // global config.

        return mAtm.mRootActivityContainer;

    }



    @HotPath(caller = HotPath.PROCESS_CHANGE)

services/core/java/com/android/server/wm/WindowState.java

+1 −1
Original line number Diff line number Diff line
@@ -3104,7 +3104,7 @@ class WindowState extends WindowContainer<WindowState> implements WindowManagerP 
            if (DEBUG_RESIZE || DEBUG_ORIENTATION) Slog.v(TAG, "Reporting new frame to " + this

                    + ": " + mWindowFrames.mCompatFrame);

            final MergedConfiguration mergedConfiguration =

                    new MergedConfiguration(mWmService.mRoot.getConfiguration(),

                    new MergedConfiguration(getProcessGlobalConfiguration(),

                    getMergedOverrideConfiguration());



            setLastReportedMergedConfiguration(mergedConfiguration);

services/tests/wmtests/src/com/android/server/wm/WindowProcessControllerTests.java

+22 −0
Original line number Diff line number Diff line
@@ -26,7 +26,9 @@ import static org.junit.Assert.assertTrue; 
import static org.mockito.Mockito.mock;



import android.content.pm.ApplicationInfo;

import android.content.res.Configuration;

import android.platform.test.annotations.Presubmit;

import android.view.DisplayInfo;



import org.junit.Test;
@@ -78,6 +80,26 @@ public class WindowProcessControllerTests extends ActivityTestsBase { 
        assertEquals(INVALID_DISPLAY, wpc.getDisplayId());

    }



    @Test

    public void testConfigurationForSecondaryScreen() {

        final WindowProcessController wpc = new WindowProcessController(

                mService, mock(ApplicationInfo.class), null, 0, -1, null, null);

        //By default, the process should not listen to any display.

        assertEquals(INVALID_DISPLAY, wpc.getDisplayId());



        // Register to a new display as a listener.

        final DisplayInfo info = new DisplayInfo();

        info.logicalWidth = 100;

        info.logicalHeight = 100;

        TestActivityDisplay display = addNewActivityDisplayAt(info, WindowContainer.POSITION_TOP);

        wpc.registerDisplayConfigurationListenerLocked(display);



        assertEquals(display.mDisplayId, wpc.getDisplayId());

        final Configuration expectedConfig = mService.mRootActivityContainer.getConfiguration();

        expectedConfig.updateFrom(display.getConfiguration());

        assertEquals(expectedConfig, wpc.getConfiguration());

    }



    private TestActivityDisplay createTestActivityDisplayInContainer() {

        final TestActivityDisplay testActivityDisplay = createNewActivityDisplay();

        mRootActivityContainer.addChild(testActivityDisplay, POSITION_TOP);