Revert "Add/plumb SurfaceControl.DISPLAY_DECORATION"

Revert "Add eLayerIsDisplayDecoration flag"

Revert submission 16511727-wm_DISPLAY_DECORATION

Reason for revert: b/211835607
Reverted Changes:
I1da3199ba:Allow changing composition from DISPLAY_DECORATION...
I1fae74b36:Update documentation for DISPLAY_DECORATION
I1da03a88f:Add eLayerIsDisplayDecoration flag
I7f22dfd03:Add/plumb SurfaceControl.DISPLAY_DECORATION

Change-Id: Ieef6e12206aaab6d3bcd2819a139ae1f201b2f8d

2 files changed, 0 insertions, 47 deletions

diff --git a/core/java/android/view/SurfaceControl.java b/core/java/android/view/SurfaceControl.java
index ab33feae7c56..3b5270960c99 100644
--- a/core/java/android/view/SurfaceControl.java
+++ b/core/java/android/view/SurfaceControl.java
@@ -515,15 +515,6 @@ public final class SurfaceControl implements Parcelable {
     public static final int ENABLE_BACKPRESSURE = 0x00000100;
 
     /**
-     * Buffers from this SurfaceControl should be considered display decorations.
-     *
-     * If the hardware has optimizations for display decorations (e.g. rounded corners, camera
-     * cutouts, etc), it should use them for this layer.
-     * @hide
-     */
-    public static final int DISPLAY_DECORATION = 0x00000200;
-
-    /**
      * Surface creation flag: Creates a surface where color components are interpreted
      * as "non pre-multiplied" by their alpha channel. Of course this flag is
      * meaningless for surfaces without an alpha channel. By default
@@ -3275,21 +3266,6 @@ public final class SurfaceControl implements Parcelable {
         }
 
         /**
-         * Sets whether the surface should take advantage of display decoration optimizations.
-         * @hide
-         */
-        public Transaction setDisplayDecoration(SurfaceControl sc, boolean displayDecoration) {
-            checkPreconditions(sc);
-            if (displayDecoration) {
-                nativeSetFlags(mNativeObject, sc.mNativeObject, DISPLAY_DECORATION,
-                        DISPLAY_DECORATION);
-            } else {
-                nativeSetFlags(mNativeObject, sc.mNativeObject, 0, DISPLAY_DECORATION);
-            }
-            return this;
-        }
-
-        /**
          * Indicates whether the surface must be considered opaque, even if its pixel format is
          * set to translucent. This can be useful if an application needs full RGBA 8888 support
          * for instance but will still draw every pixel opaque.
diff --git a/core/java/android/view/ViewRootImpl.java b/core/java/android/view/ViewRootImpl.java
index 810919710550..c37120220f1a 100644
--- a/core/java/android/view/ViewRootImpl.java
+++ b/core/java/android/view/ViewRootImpl.java
@@ -491,9 +491,6 @@ public final class ViewRootImpl implements ViewParent,
     protected final ViewFrameInfo mViewFrameInfo = new ViewFrameInfo();
     private final InputEventAssigner mInputEventAssigner = new InputEventAssigner();
 
-    // Whether to draw this surface as DISPLAY_DECORATION.
-    boolean mDisplayDecorationCached = false;
-
     /**
      * Update the Choreographer's FrameInfo object with the timing information for the current
      * ViewRootImpl instance. Erase the data in the current ViewFrameInfo to prepare for the next
@@ -2845,9 +2842,6 @@ public final class ViewRootImpl implements ViewParent,
                 if (mSurfaceControl.isValid()) {
                     updateOpacity(mWindowAttributes, dragResizing,
                             surfaceControlChanged /*forceUpdate */);
-                    if (surfaceControlChanged) {
-                        updateDisplayDecoration();
-                    }
                 }
 
                 if (DEBUG_LAYOUT) Log.v(mTag, "relayout: frame=" + frame.toShortString()
@@ -10405,23 +10399,6 @@ public final class ViewRootImpl implements ViewParent,
     }
 
     /**
-     * @hide
-     */
-    public void setDisplayDecoration(boolean displayDecoration) {
-        if (displayDecoration == mDisplayDecorationCached) return;
-
-        mDisplayDecorationCached = displayDecoration;
-
-        if (mSurfaceControl.isValid()) {
-            updateDisplayDecoration();
-        }
-    }
-
-    private void updateDisplayDecoration() {
-        mTransaction.setDisplayDecoration(mSurfaceControl, mDisplayDecorationCached).apply();
-    }
-
-    /**
      * Sends a list of blur regions to SurfaceFlinger, tagged with a frame.
      *
      * @param regionCopy List of regions