Revert "[css-anchor-position] First step of scroll adjustment with overflow:hidden"
This reverts commit b7083805b9cf8a166f46624b98bef4ea5a72fe06.
Reason for revert: The CL caused crbug.com/337909142.
On composited scrolls, when we update the main thread scroll offset,
we normally omit commit, and the changed default adjustment is not
committed, and the anchor adjustment will be incorrect if anything
causes (e.g. a non-scrollable scroll container without a scroll node)
fallback to the default adjustment.
Original change's description:
[css-anchor-position] First step of scroll adjustment with overflow:hidden
This implements fail-safe for missing nodes referenced from
adjustment ids of an anchored transform node. The case may happen
- for overflow:hidden scrollers for which we don't create scroll nodes
for now;- for adjustment containers that are not painted due to cull rect,
etc.We'll use the main-thread calculated adjustment when any node is
missing. This is not perfect (e.g. position of anchored may lag behind
composited scrolling) but is better than not being anchored or jumping
between anchored and not anchored.We may implement better solutions and decide whether to remove this
fail-safe (or convert it to CHECK) in the future.Bug: 332331854
Change-Id: I9b9121a173fdc52eeaf2987f3886155f3ea773bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5455174
Reviewed-by: Philip Rogers pdr@chromium.org
Commit-Queue: Xianzhu Wang wangxianzhu@chromium.org
Cr-Commit-Position: refs/heads/main@{#1288314}
Bug: 332331854, 337909142
Change-Id: I5528aeee1ace8dfff0d6cf299941ede6a4961a3b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5503090
Commit-Queue: Xianzhu Wang wangxianzhu@chromium.org
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Reviewed-by: Philip Rogers pdr@chromium.org
Commit-Queue: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Commit-Queue: Philip Rogers pdr@chromium.org
Auto-Submit: Xianzhu Wang wangxianzhu@chromium.org
Cr-Commit-Position: refs/heads/main@{#1294594}