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

Unified Diff: LayoutTests/compositing/squashing/squashed-layer-loses-graphicslayer.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/squashing/squashed-layer-loses-graphicslayer.html
diff --git a/LayoutTests/compositing/squashing/squashed-layer-loses-graphicslayer.html b/LayoutTests/compositing/squashing/squashed-layer-loses-graphicslayer.html
index 7053ad0c80480bbfa64e2175262f8aff0942cdb0..3f3ccbab1ce4108081eb2999e12184a7828c60c9 100644
--- a/LayoutTests/compositing/squashing/squashed-layer-loses-graphicslayer.html
+++ b/LayoutTests/compositing/squashing/squashed-layer-loses-graphicslayer.html
@@ -1,6 +1,7 @@
<!DOCTYPE html>
<html>
<head>
+<script src="../../resources/run-after-display.js"></script>
<style>
div {
position: absolute;
@@ -52,8 +53,10 @@ div:hover {
</style>
<script>
- if (window.testRunner)
+ if (window.testRunner) {
testRunner.dumpAsText();
+ testRunner.waitUntilDone();
+ }
if (window.internals)
internals.settings.setLayerSquashingEnabled(true);
@@ -62,9 +65,11 @@ div:hover {
{
if (!window.internals)
return;
+ runAfterDisplay(executeTestCases);
+ }
- testRunner.display();
-
+ function executeTestCases()
+ {
// Case 1
document.getElementById('Case1').textContent = window.internals.layerTreeAsText(document, internals.LAYER_TREE_INCLUDES_REPAINT_RECTS);
@@ -76,6 +81,9 @@ div:hover {
// Display the test results only after test is done so that it does not affect repaint rect results.
document.getElementById('testResults').style.display = "block";
+
+ if (window.testRunner)
+ testRunner.notifyDone();
}
</script>
</head>

Powered by Google App Engine
This is Rietveld 408576698