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

Unified Diff: LayoutTests/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html

Issue 196653012: Remove display() from compositing/ LayoutTests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: softwarecompositing expectations 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/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html
diff --git a/LayoutTests/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html b/LayoutTests/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html
index cffc66b27025342c093b3b98ed9fc3aa34b3fd73..c2e4323d020100c8dd9526a8a3b14fd64cef2e6a 100644
--- a/LayoutTests/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html
+++ b/LayoutTests/compositing/iframes/fixed-position-element-in-iframe-enters-viewport.html
@@ -15,6 +15,7 @@ If this test passes, you should see a green square.
border: 0;
}
</style>
+ <script src="../../resources/run-after-display.js"></script>
<script>
if (window.testRunner)
testRunner.waitUntilDone();
@@ -28,18 +29,15 @@ If this test passes, you should see a green square.
{
// Initially, the fixed position element in the iframe will not
// paint because the iframe has zero height.
- if (window.testRunner)
- testRunner.display();
+ runAfterDisplay(function() {
+ // Give the iframe a non-zero height. The fixed position element
+ // inside the iframe should paint now.
+ var iframeElement = document.getElementById("iframe-containing-fixed-position-element");
+ iframeElement.style.height = "100px";
- // Give the iframe a non-zero height. The fixed position element
- // inside the iframe should paint now.
- var iframeElement = document.getElementById("iframe-containing-fixed-position-element");
- iframeElement.style.height = "100px";
-
- if (window.testRunner) {
- testRunner.display();
- testRunner.notifyDone();
- }
+ if (window.testRunner)
+ testRunner.notifyDone();
+ });
}
</script>
</head>

Powered by Google App Engine
This is Rietveld 408576698