Commit 0ef8e04d authored by Xianzhu Wang's avatar Xianzhu Wang Committed by Commit Bot

Revert "Reenable scroll-animations/element-based-offset tests"

This reverts commit bc2f9ba0.

Reason for revert: The test is still flaky.

Original change's description:
> Reenable scroll-animations/element-based-offset tests
>
> The underlying source of the test flakes was likely addressed in
> recent changes to the handling of play and pause for scroll-linked
> animations.
>
> https://chromium-review.googlesource.com/c/chromium/src/+/2386339
>
> This patch also contains a cleanup to remove the unnecessarily
> convoluted way of hiding the scrollbar for the clamped offset test.
>
> Tested with 1000+ retries of the two tests in a virtual threaded
> environment.
>
> Bug: 1080609
> Change-Id: If9097b0a2017a9c87a359b61ba93f35bae56b512
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2475935
> Reviewed-by: Xida Chen <xidachen@chromium.org>
> Commit-Queue: Kevin Ellis <kevers@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#817568}

TBR=kevers@chromium.org,xidachen@chromium.org

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

Bug: 1080609
Change-Id: Ic31eede5561182d78e7bf805cd1f112edfd5379c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483585Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#818355}
parent af4e791e
...@@ -145,7 +145,7 @@ virtual/composite-after-paint/paint/frames/* [ Pass ] ...@@ -145,7 +145,7 @@ virtual/composite-after-paint/paint/frames/* [ Pass ]
virtual/composite-after-paint/scrollingcoordinator/* [ Pass ] virtual/composite-after-paint/scrollingcoordinator/* [ Pass ]
# --- End CompositeAfterPaint Tests -- # --- End CompositeAfterPaint Tests --
# --- BEGIN OOP-R Canvas tests --- # --- BEGIN OOP-R Canvas tests ---
crbug.com/1081534 [ Win7 ] virtual/oopr-canvas2d/fast/canvas/canvas-composite-video-shadow.html [ Pass Failure ] crbug.com/1081534 [ Win7 ] virtual/oopr-canvas2d/fast/canvas/canvas-composite-video-shadow.html [ Pass Failure ]
crbug.com/1081534 [ Win ] virtual/oopr-canvas2d/fast/canvas/canvas-clearRect-first.html [ Failure ] crbug.com/1081534 [ Win ] virtual/oopr-canvas2d/fast/canvas/canvas-clearRect-first.html [ Failure ]
crbug.com/1081534 [ Win ] virtual/oopr-canvas2d/fast/canvas/canvas-clearRect-with-clip.html [ Failure ] crbug.com/1081534 [ Win ] virtual/oopr-canvas2d/fast/canvas/canvas-clearRect-with-clip.html [ Failure ]
...@@ -6140,6 +6140,9 @@ crbug.com/1011811 http/tests/devtools/network/network-xhr-data-received-async-re ...@@ -6140,6 +6140,9 @@ crbug.com/1011811 http/tests/devtools/network/network-xhr-data-received-async-re
crbug.com/1011811 http/tests/devtools/network/download.js [ Pass Failure ] crbug.com/1011811 http/tests/devtools/network/download.js [ Pass Failure ]
crbug.com/1011811 http/tests/devtools/sxg/sxg-disable-cache.js [ Pass Failure ] crbug.com/1011811 http/tests/devtools/sxg/sxg-disable-cache.js [ Pass Failure ]
crbug.com/1080609 virtual/threaded/external/wpt/scroll-animations/element-based-offset.html [ Pass Failure ]
crbug.com/1080609 virtual/threaded/external/wpt/scroll-animations/element-based-offset-clamp.html [ Pass Failure ]
# Fails when test moved to use full compositor pipe. # Fails when test moved to use full compositor pipe.
crbug.com/1085832 [ Fuchsia ] images/size-failure.html [ Timeout ] crbug.com/1085832 [ Fuchsia ] images/size-failure.html [ Timeout ]
......
...@@ -7,19 +7,25 @@ ...@@ -7,19 +7,25 @@
<script src="testcommon.js"></script> <script src="testcommon.js"></script>
<style> <style>
/*
* Overflow hidden prevents user scroll including mouse wheel; however, the
* element is still a scrollable container and can be scrolled programmatically.
* Removing the visible scrollbars in this manner simplifies the position
* calculations in the text.
*/
.scroller { .scroller {
overflow: hidden; overflow: scroll;
height: 500px; height: 500px;
width: 500px; width: 500px;
will-change: transform; will-change: transform;
} }
/* Disable scrollbars to simplify the calculations in the test. */
.scroller {
scrollbar-width: 0;
}
/*
Chrome does not support scrollbar-width so we use this non-standard property
until it does.
*/
.scroller::-webkit-scrollbar {
display: none;
}
.contents { .contents {
height: 1200px; height: 1200px;
width: 1200px; width: 1200px;
...@@ -206,4 +212,4 @@ ...@@ -206,4 +212,4 @@
createScrollAnimationTest(description, config); createScrollAnimationTest(description, config);
} }
} }
</script> </script>
\ No newline at end of file
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment