Bug 1760890 [wpt PR 33308] - Revert "Create property tree nodes for will-change only...
commit4f0a30a546a11ba7e3016e22189d93ba4d685788
authorDavid Baron <dbaron@chromium.org>
Thu, 24 Mar 2022 12:19:50 +0000 (24 12:19 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Sat, 26 Mar 2022 21:58:18 +0000 (26 21:58 +0000)
treeed767a55f0416f5199f41e831d572f026d12b628
parentf7cc6460c081ca50fe91ca9b46b8c6aa5e3f09e8
Bug 1760890 [wpt PR 33308] - Revert "Create property tree nodes for will-change only when relevant property has will-change.", a=testonly

Automatic update from web-platform-tests
Revert "Create property tree nodes for will-change only when relevant property has will-change."

This reverts commit 5d3dd81ee4f41ffc7fd40fda745cb097cd50e9f5.

Reason for revert: failure of fast/sub-pixel/transformed-iframe-copy-on-scroll.html on mac-mac11-arm64.

Original change's description:
> Create property tree nodes for will-change only when relevant property has will-change.
>
> Prior to this change, will-change: transform, opacity, filter, or
> backdrop-filter would cause the creation of property tree nodes (in both
> the paint property tree and the cc property tree) for all of them
> (though only a single effect property node for both opacity and
> backdrop-filter).
>
> With this change, will-change: transform only causes the creation of a
> transform node (in the transform tree), will-change: filter only causes
> the creation of a filter node (in the effect tree), and will-change:
> opacity or backdrop-filter only causes the creation of an effect node
> (in the effect tree).  However, *if* the nodes are created by something
> else, the presence of a different will-change still causes any node that
> is created in the paint property tree to be composited (i.e., created in
> the cc property tree).
>
> This provides the basis on which we can separate the transform node into
> multiple transform nodes, for efficient implementation and animation of
> individual transform properties, without causing a performance
> regression.
>
> Earlier versions of this CL (prior to the change that kept nodes being
> composited for an unrelated will-change) caused the test
> external/wpt/css/filter-effects/effect-reference-feimage-002.html to
> start failing (in the same way as -001 and -003) because the filter
> effect is no longer composited.  It creates a copy of the test as -004
> with will-change: filter rather than will-change: transform that
> continues to have a composited filter effect node and thus continues
> passing even with that earlier change.  This test still seems worth
> adding despite no longer strictly being needed.
>
> Bug: 900241
> Change-Id: I18efc630668ea7eac2a8c0a0417b766762e71d9f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3328730
> Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
> Commit-Queue: David Baron <dbaron@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#983821}

Bug: 900241
Change-Id: I8143ff91fa0d856ab189414e44f12ce1bcdf77e8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3544106
Auto-Submit: David Baron <dbaron@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: David Baron <dbaron@chromium.org>
Commit-Queue: David Baron <dbaron@chromium.org>
Cr-Commit-Position: refs/heads/main@{#984017}

--

wpt-commits: a96c12ebc2df6ca08a8c94f847d4b78d20ed6117
wpt-pr: 33308
testing/web-platform/tests/css/filter-effects/effect-reference-feimage-004.html [deleted file]
testing/web-platform/tests/css/filter-effects/reference/effect-reference-feimage-004-ref.html [deleted file]