LayoutTests/css3/filters/multiple-filters-invalidation.html - Issue 196573042: Remove display() from more layout tests

Unified Diff: LayoutTests/css3/filters/multiple-filters-invalidation.html

Issue 196573042: Remove display() from more layout tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Remove video-frame-size-change test 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:

Index: LayoutTests/css3/filters/multiple-filters-invalidation.html

diff --git a/LayoutTests/css3/filters/multiple-filters-invalidation.html b/LayoutTests/css3/filters/multiple-filters-invalidation.html

index 5b830051d9b48e51179715b7dfb7046550eacd7e..08008e8758efea875647d8b0d27f9eb97d93e8b7 100644

--- a/LayoutTests/css3/filters/multiple-filters-invalidation.html

+++ b/LayoutTests/css3/filters/multiple-filters-invalidation.html

@@ -14,17 +14,19 @@ html {

<body>

</body>

+<script src="../../resources/run-after-display.js"></script>

ojan 2014/03/27 20:31:49 Should run-after-display.js call testRunner.waitUn

enne (OOO) 2014/03/27 20:51:05 It's a good point. I think maybe I didn't like th

ojan 2014/03/27 21:59:54 Yeah, I don't like js files doing weird setup eith

function changeBackground() {

document.getElementById("rect").style.background = "green";

+ if (window.testRunner) {

+ testRunner.notifyDone();

+ }

}

if (window.testRunner) {

testRunner.dumpAsTextWithPixelResults();

- testRunner.display();

- changeBackground();

-} else

- window.setTimeout(changeBackground, 250);

+ testRunner.waitUntilDone();

+runAfterDisplay(changeBackground);

</script>

</html>