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

Unified Diff: LayoutTests/compositing/overflow/repaint-after-losing-scrollbars.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/overflow/repaint-after-losing-scrollbars.html
diff --git a/LayoutTests/compositing/overflow/repaint-after-losing-scrollbars.html b/LayoutTests/compositing/overflow/repaint-after-losing-scrollbars.html
index 50dba35bfc297a8d5f2190f8f63eb6abee69081c..56b064666b5bf8585b5ac036997684547e198f64 100644
--- a/LayoutTests/compositing/overflow/repaint-after-losing-scrollbars.html
+++ b/LayoutTests/compositing/overflow/repaint-after-losing-scrollbars.html
@@ -1,5 +1,6 @@
<!DOCTYPE html>
<!-- there should be four green squares and no visible red -->
+<script src="../../resources/run-after-display.js"></script>
<style>
.container {
width: 100px;
@@ -57,12 +58,14 @@ function finishTest() {
document.getElementById("both").classList.remove("wide"); // third div loses both
document.getElementById("both").classList.remove("tall"); // vertical and horizontal scrollbars
document.getElementById("corner").classList.remove("resizeWidget"); // bottom div loses a resize corner but no scrollbars
+ if (window.testRunner)
+ testRunner.notifyDone();
}
if (window.testRunner) {
testRunner.dumpAsTextWithPixelResults();
- testRunner.display(); // Paint once with all overflow controls visible.
- finishTest();
-} else
- window.setTimeout(finishTest, 0);
+ testRunner.waitUntilDone();
+}
+
+runAfterDisplay(finishTest);
</script>

Powered by Google App Engine
This is Rietveld 408576698