Chromium Code Reviews Chromium Code Reviews
Issue 191693002:
Delay scrollContents until the next paint (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| Index: LayoutTests/fast/repaint/scroll-in-fixed-layer.html |
| diff --git a/LayoutTests/fast/repaint/scroll-in-fixed-layer.html b/LayoutTests/fast/repaint/scroll-in-fixed-layer.html |
| index 46301817787d4dac6a926e535c48a8f78faa1301..4db48c6f91b2835358a44689f3bb297479f5f117 100644 |
| --- a/LayoutTests/fast/repaint/scroll-in-fixed-layer.html |
| +++ b/LayoutTests/fast/repaint/scroll-in-fixed-layer.html |
| @@ -1,10 +1,23 @@ |
| <html> |
| <head> |
| <link rel="stylesheet" href="resources/default.css"></style> |
| + <script src="../../resources/run-after-display.js"></script> |
| <script src="resources/text-based-repaint.js" type="text/javascript"></script> |
| <script type="text/javascript"> |
| - if (window.testRunner) |
| + if (window.testRunner) { |
| + testRunner.waitUntilDone(); |
| testRunner.dumpAsTextWithPixelResults(); |
| + } |
| + |
| + function scrollAndRepaint() |
|
esprehn
2014/03/24 21:50:43
Yeah can we just switch to onload = for all these?
ykyyip
2014/03/24 22:15:03
Done.
|
| + { |
| + window.scrollTo(1000, 1000); |
| + runAfterDisplay(function() { |
| + runRepaintTest(); |
| + if (window.testRunner) |
| + testRunner.notifyDone(); |
| + }) |
| + } |
| function repaintTest() |
| { |
| @@ -14,7 +27,7 @@ |
| } |
| </script> |
| </head> |
| -<body style="height:2000px;" onload="runRepaintTest()"> |
| +<body style="height:2000px;" onload="scrollAndRepaint()"> |
| <!-- You should see 1 green rectangle in the output and no red. --> |
| <div id="moveMe" style="top: 150px; left: 100px;" class="fixed clipped"> |
| <div class="absolute green" style="top: 100px; left: 150px;"></div> |
| @@ -22,8 +35,5 @@ |
| </div> |
| <!-- Make sure we are scrolled --> |
| <div style="top: 0px; left: 0px;" class="absolute red"></div> |
| - <script> |
| - window.scrollTo(1000, 1000); |
| - </script> |
| </body> |
| </html> |
