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

Unified Diff: LayoutTests/compositing/iframes/resources/fixed-position-subframe.html

Issue 181653006: Remove testRunner.display() from most compositing/ tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Revert change to plugin test Created 6 years, 10 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/resources/fixed-position-subframe.html
diff --git a/LayoutTests/compositing/iframes/resources/fixed-position-subframe.html b/LayoutTests/compositing/iframes/resources/fixed-position-subframe.html
index 5dc2d1c73a10b922fe63c42365d181c350c1b1dc..7c92bb48c254d315f067e97c941f32e82cc27ecb 100644
--- a/LayoutTests/compositing/iframes/resources/fixed-position-subframe.html
+++ b/LayoutTests/compositing/iframes/resources/fixed-position-subframe.html
@@ -1,5 +1,6 @@
<html>
<head>
+<script src="../../../resources/run-after-display.js"></script>
<style>
body {
height: 5000px;
@@ -23,12 +24,16 @@
</head>
<script>
function doTest() {
- if (window.testRunner)
+ if (window.testRunner) {
testRunner.dumpAsTextWithPixelResults();
+ testRunner.waitUntilDone();
+ }
window.scrollTo(0, 2000);
- if (window.testRunner)
- testRunner.display();
- window.scrollTo(0, 1800);
+ runAfterDisplay(function() {
+ window.scrollTo(0, 1800);
+ if (window.testRunner)
+ testRunner.notifyDone();
+ });
}
window.addEventListener("load", doTest, false);
</script>

Powered by Google App Engine
This is Rietveld 408576698