Chromium Code Reviews Chromium Code Reviews
Issue 714083003:
Revert "Move some scroll invalidations to the paint invalidation phase" (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/repaint/scroll-stacking-context-backface-visiblity-leaves-traces.html |
| diff --git a/LayoutTests/fast/repaint/scroll-stacking-context-backface-visiblity-leaves-traces.html b/LayoutTests/fast/repaint/scroll-stacking-context-backface-visiblity-leaves-traces.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..106fcf6e6bc11db2af7a8d3c1fbf508eff3d9cec |
| --- /dev/null |
| +++ b/LayoutTests/fast/repaint/scroll-stacking-context-backface-visiblity-leaves-traces.html |
| @@ -0,0 +1,55 @@ |
| +<!DOCTYPE html> |
| +<style> |
| +#searchbar { |
| + bottom: 0; /* Useful for seeing the issue visually. */ |
| + position: fixed; |
| +} |
| +#recentlink { |
| + background: purple; |
| + position: absolute; |
| + width: 150px; |
| + height: 150px; |
| +} |
| +header { |
| + padding: 10px; |
| + position: relative; |
| + z-index: 50; |
| + -webkit-backface-visibility: hidden; |
| +} |
| +</style> |
| +To manually test, just scroll down, there should be no purple trace. |
| +<header> |
| + <div id="searchbar"> |
| + <div id="recentlink"></div> |
| + </div> |
| +</header> |
| + |
| +<!-- Used to have some overflowing content to scroll --> |
| +<div style="height: 5000px"></div> |
| + |
| +<script src="resources/text-based-repaint.js" type="text/javascript"></script> |
| +<script> |
| +window.scrollTo(0, 1000); |
| + |
| +window.testIsAsync = true; |
| +if (window.testRunner) |
| + testRunner.waitUntilDone(); |
| + |
| +function repaintTest() |
| +{ |
| + window.scrollBy(0, 400); |
| + finishRepaintTest(); |
| +} |
| +function showBug() |
| +{ |
| + window.requestAnimationFrame(function() { |
| + window.requestAnimationFrame(function() { |
| + runRepaintTest(); |
| + }); |
| + }); |
| +} |
| +showBug(); |
| +//window.addEventListener("load", showBug, false); |
|
dsinclair
2014/11/11 19:51:23
remove?
Julien - ping for review
2014/11/12 18:46:01
Indeed, cleaned up the test (showBug() wasn't usef
|
| + |
| +</script> |
| + |
