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

Commit f6ef1833 authored by Jorim Jaggi's avatar Jorim Jaggi
Browse files

Revert "Make WindowContainer surfaces container layers by default." 

This reverts commit 139c7776.

Reason: Device gets put into GL comp pretty much all the time,
trashing performance and battery.

Bug: 123686354
Bug: 122561221
Change-Id: Icf658f331c407d03e844557cc2531c034aa38083
Exempt-From-Owner-Approval: Simple revert
parent c7026dd6

core/java/android/app/ActivityView.java

+1 −1
Original line number Diff line number Diff line
@@ -363,7 +363,7 @@ public class ActivityView extends ViewGroup { 
        final IWindowManager wm = WindowManagerGlobal.getWindowManagerService();



        mRootSurfaceControl = new SurfaceControl.Builder(surfaceSession)

                .setContainerLayer()

                .setContainerLayer(true)

                .setParent(mSurfaceView.getSurfaceControl())

                .setName(DISPLAY_NAME)

                .build();

core/java/android/view/SurfaceControl.java

+18 −22
Original line number Diff line number Diff line
@@ -501,16 +501,7 @@ public final class SurfaceControl implements Parcelable { 
            }

            mWidth = width;

            mHeight = height;

            // set this as a buffer layer since we are specifying a buffer size.

            return setFlags(FX_SURFACE_NORMAL, FX_SURFACE_MASK);

        }



        /**

         * Set the initial size of the controlled surface's buffers in pixels.

         */

        private void unsetBufferSize() {

            mWidth = 0;

            mHeight = 0;

            return this;

        }



        /**
@@ -628,11 +619,16 @@ public final class SurfaceControl implements Parcelable { 
         * Color layers will not have an associated BufferQueue and will instead always render a

         * solid color (that is, solid before plane alpha). Currently that color is black.

         *

         * @param isColorLayer Whether to create a color layer.

         * @hide

         */

        public Builder setColorLayer() {

            unsetBufferSize();

            return setFlags(FX_SURFACE_DIM, FX_SURFACE_MASK);

        public Builder setColorLayer(boolean isColorLayer) {

            if (isColorLayer) {

                mFlags |= FX_SURFACE_DIM;

            } else {

                mFlags &= ~FX_SURFACE_DIM;

            }

            return this;

        }



        private boolean isColorLayerSet() {
@@ -645,11 +641,16 @@ public final class SurfaceControl implements Parcelable { 
         * Container layers will not be rendered in any fashion and instead are used

         * as a parent of renderable layers.

         *

         * @param isContainerLayer Whether to create a container layer.

         * @hide

         */

        public Builder setContainerLayer() {

            unsetBufferSize();

            return setFlags(FX_SURFACE_CONTAINER, FX_SURFACE_MASK);

        public Builder setContainerLayer(boolean isContainerLayer) {

            if (isContainerLayer) {

                mFlags |= FX_SURFACE_CONTAINER;

            } else {

                mFlags &= ~FX_SURFACE_CONTAINER;

            }

            return this;

        }



        private boolean isContainerLayerSet() {
@@ -657,7 +658,7 @@ public final class SurfaceControl implements Parcelable { 
        }



        /**

         * Set 'Surface creation flags' such as {@link #HIDDEN}, {@link #SECURE}.

         * Set 'Surface creation flags' such as {@link HIDDEN}, {@link SECURE}.

         *

         * TODO: Finish conversion to individual builder methods?

         * @param flags The combined flags
@@ -667,11 +668,6 @@ public final class SurfaceControl implements Parcelable { 
            mFlags = flags;

            return this;

        }



        private Builder setFlags(int flags, int mask) {

            mFlags = (mFlags & ~mask) | flags;

            return this;

        }

    }



    /**

core/java/android/view/SurfaceView.java

+2 −1
Original line number Diff line number Diff line
@@ -33,6 +33,7 @@ import android.graphics.Region; 
import android.graphics.RenderNode;

import android.os.Build;

import android.os.Handler;

import android.os.IBinder;

import android.os.Looper;

import android.os.SystemClock;

import android.util.AttributeSet;
@@ -587,7 +588,7 @@ public class SurfaceView extends View implements ViewRootImpl.WindowStoppedCallb 
                    mBackgroundControl = new SurfaceControl.Builder(mSurfaceSession)

                        .setName("Background for -" + name)

                        .setOpaque(true)

                        .setColorLayer()

                        .setColorLayer(true)

                        .setParent(mSurfaceControl)

                        .build();

services/core/java/com/android/server/display/ColorFade.java

+1 −1
Original line number Diff line number Diff line
@@ -575,7 +575,7 @@ final class ColorFade { 
                final SurfaceControl.Builder builder =

                        new SurfaceControl.Builder(mSurfaceSession).setName("ColorFade");

                if (mMode == MODE_FADE) {

                    builder.setColorLayer();

                    builder.setColorLayer(true);

                } else {

                    builder.setBufferSize(mDisplayWidth, mDisplayHeight);

                }

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

+1 −1
Original line number Diff line number Diff line
@@ -48,7 +48,7 @@ public class BlackFrame { 


            surface = dc.makeOverlay()

                    .setName("BlackSurface")

                    .setColorLayer()

                    .setColorLayer(true)

                    .setParent(null) // TODO: Work-around for b/69259549

                    .build();

            transaction.setWindowCrop(surface, w, h);