Revert "Use composited scrolling if resizer needs to be above descendants"

This reverts commit 7f72fc62.

Reason for revert: Caused P0 crbug.com/1155271.

Original change's description:
> Use composited scrolling if resizer needs to be above descendants
>
> A resizer on an element containing composited stacked descendants
> needs to be painted after the descendants. In pre-CompositeAfterPaint
> this can only be achieved by using composited scrolling for the element
> so that we'll composite the resizer even if the element is not
> scrollable.
>
> In CompositeAfterPaint everything is good because we always use the
> correct paint order.
>
> Bug: 1135676
> Change-Id: Ic69b830ae3d71cd9f6b8f55deca591b485b189a5
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523748
> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
> Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#825613}

TBR=wangxianzhu@chromium.org,chrishtr@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1135676
Change-Id: Ie47558d1e9cbb1ed8f29e21286128899085be338
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2583603Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835581}

third_party/blink/renderer/core/paint/compositing/compositing_inputs_updater.cc

@@ -219,6 +219,7 @@ void CompositingInputsUpdater::UpdateSelfAndDescendantsRecursively(
       descendant_has_direct_compositing_reason);
 
   if ((layer->IsRootLayer() || layer->NeedsReorderOverlayOverflowControls()) &&
+      layer->ScrollsOverflow() &&
       layer->DescendantHasDirectOrScrollingCompositingReason() &&
       !layer->NeedsCompositedScrolling())
     layer->GetScrollableArea()->UpdateNeedsCompositedScrolling(true);
@@ -367,8 +368,7 @@ void CompositingInputsUpdater::UpdateAncestorInfo(PaintLayer* const layer,
   // <div style="overflow:scroll;">
   //   <div style="position:relative;">Paint sibling.</div>
   // </div>
-  if (layer->ScrollsOverflow() ||
-      layer->NeedsReorderOverlayOverflowControls()) {
+  if (layer->ScrollsOverflow()) {
     info.scrolling_ancestor = layer;
     info.needs_reparent_scroll = true;
   }

third_party/blink/renderer/core/paint/paint_layer_scrollable_area.cc

@@ -2565,8 +2565,7 @@ bool PaintLayerScrollableArea::ComputeNeedsCompositedScrollingInternal(
   if (CompositingReasonFinder::RequiresCompositingForRootScroller(*layer_))
     return true;
 
-  if (!layer_->ScrollsOverflow() &&
-      !layer_->NeedsReorderOverlayOverflowControls())
+  if (!layer_->ScrollsOverflow())
     return false;
 
   if (layer_->Size().IsEmpty())

third_party/blink/web_tests/FlagExpectations/composite-after-paint

@@ -38,6 +38,7 @@ crbug.com/802915 css3/blending/isolation-should-include-non-local-background.htm
 crbug.com/918155 virtual/prefer_compositing_to_lcd_text/scrollbars/overlay-scrollbar-over-child-layer-nested-2.html [ Pass ]
 crbug.com/918155 virtual/prefer_compositing_to_lcd_text/scrollbars/overlay-scrollbar-over-child-layer-nested.html [ Pass ]
 paint/invalidation/compositing/subpixel-offset-scaled-transform-composited.html [ Pass ]
+crbug.com/1135676 external/wpt/css/css-ui/resize-child-will-change-transform.html [ Pass ]
 
 # Raster invalidation doesn't work for huge layers.
 paint/invalidation/raster-under-invalidation-checking.html [ Failure ]

third_party/blink/web_tests/TestExpectations

@@ -425,6 +425,9 @@ crbug.com/1050993 [ Linux ] virtual/gpu-rasterization/images/drag-image-descenda
 # Flaky test.
 crbug.com/1054894 [ Mac ] http/tests/images/image-decode-in-frame.html [ Pass Failure ]
 
+# Fixed in CompositeAfterPaint.
+crbug.com/1135676 external/wpt/css/css-ui/resize-child-will-change-transform.html [ Failure ]
+
 # ====== Paint team owned tests to here ======
 
 crbug.com/1142958 external/wpt/layout-instability/absolute-child-shift-with-parent-will-change.html [ Failure ]

third_party/blink/web_tests/flag-specific/disable-layout-ng/paint/invalidation/position/layout-state-only-positioned-expected.txt

@@ -4,7 +4,10 @@
       "name": "Scrolling Contents Layer",
       "bounds": [800, 600],
       "contentsOpaque": true,
-      "backgroundColor": "#FFFFFF"
+      "backgroundColor": "#FFFFFF",
+      "invalidations": [
+        [0, 50, 106, 106]
+      ]
     },
     {
       "name": "LayoutBlockFlow (positioned) DIV",

third_party/blink/web_tests/paint/invalidation/position/layout-state-only-positioned-expected.txt

@@ -4,7 +4,10 @@
       "name": "Scrolling Contents Layer",
       "bounds": [800, 600],
       "contentsOpaque": true,
-      "backgroundColor": "#FFFFFF"
+      "backgroundColor": "#FFFFFF",
+      "invalidations": [
+        [0, 50, 106, 106]
+      ]
     },
     {
       "name": "LayoutNGBlockFlow (positioned) DIV",