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

Unified Diff: LayoutTests/compositing/layer-creation/should-invoke-deferred-compositing.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/layer-creation/should-invoke-deferred-compositing.html
diff --git a/LayoutTests/compositing/layer-creation/should-invoke-deferred-compositing.html b/LayoutTests/compositing/layer-creation/should-invoke-deferred-compositing.html
index 75225eac11aaa791befa55fb3f5e26b59b56af76..29292bd17c1e50273873fb954a0de273eca098ec 100644
--- a/LayoutTests/compositing/layer-creation/should-invoke-deferred-compositing.html
+++ b/LayoutTests/compositing/layer-creation/should-invoke-deferred-compositing.html
@@ -11,6 +11,7 @@
<html>
<head>
+<script src="../../resources/run-after-display.js"></script>
<style>
.composited {
@@ -62,15 +63,18 @@ canvas {
return;
}
- testRunner.display();
+ testRunner.waitUntilDone();
- // This should initiate a compositor frame via scheduleAnimation().
- addCanvasToTree();
+ runAfterDisplay(function() {
+ // This should initiate a compositor frame via scheduleAnimation().
+ addCanvasToTree();
- if (window.internals.isCompositorFramePending(document))
- document.getElementById("result").innerHTML = "PASS - did schedule animation";
- else
- document.getElementById("result").innerHTML = "FAIL - did not schedule animation";
+ if (window.internals.isCompositorFramePending(document))
+ document.getElementById("result").innerHTML = "PASS - did schedule animation";
+ else
+ document.getElementById("result").innerHTML = "FAIL - did not schedule animation";
+ testRunner.notifyDone();
+ });
}
</script>

Powered by Google App Engine
This is Rietveld 408576698