Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(792)

Unified Diff: LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html

Issue 191693002: Delay scrollContents until the next paint (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: fix layout tests Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html
diff --git a/LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html b/LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html
index 3434dcc24f9b45c52d95effd8edef8bb518786a3..3e78b61e2a89361adcac058e7f755fb80306b1b7 100644
--- a/LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html
+++ b/LayoutTests/fast/repaint/scroll-fixed-layer-with-no-visible-content.html
@@ -2,10 +2,23 @@
<head>
<title></title>
<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 same.
ykyyip 2014/03/24 22:15:03 Done.
+ {
+ window.scrollTo(1000, 1000);
+ runAfterDisplay(function() {
+ runRepaintTest();
+ if (window.testRunner)
+ testRunner.notifyDone();
+ });
+ }
function repaintTest()
{
@@ -16,13 +29,10 @@
}
</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; visibility: hidden" class="fixed clipped"><div class="absolute green" style="top: 100px; left: 150px;"></div></div>
<!-- Make sure we are scrolled -->
<div style="top: 0px; left: 0px;" class="absolute red"></div>
- <script>
- window.scrollTo(1000, 1000);
- </script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698