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

Unified Diff: sky/tests/harness/reftest-mismatch-expected-mismatch.sky

Issue 826343003: Minor improvements to reftests. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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: sky/tests/harness/reftest-mismatch-expected-mismatch.sky
diff --git a/sky/tests/harness/reftest-mismatch-expected-mismatch.sky b/sky/tests/harness/reftest-mismatch-expected-mismatch.sky
index 25e46f88ac019e580766e1026a7c26bcd2cae3b5..42f71f7abbbe993594a6d59def97ab9b47f9d80d 100644
--- a/sky/tests/harness/reftest-mismatch-expected-mismatch.sky
+++ b/sky/tests/harness/reftest-mismatch-expected-mismatch.sky
@@ -1,12 +1,9 @@
<foo>
+<import src="../resources/run-after-display.sky" as="runAfterDisplay" />
This is a mismatch reftest.
<script>
-window.addEventListener('load', function() {
- // TODO(ojan): requestAnimationFrame once we properly pump frames.
- // https://github.com/domokit/mojo/issues/32
- setTimeout(function() {
- internals.notifyTestComplete("");
- }, 100);
-})
+runAfterDisplay(function() {
+ internals.notifyTestComplete("");
+});
</script>
</foo>

Powered by Google App Engine
This is Rietveld 408576698