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

Unified Diff: LayoutTests/compositing/repaint/fixed-pos-with-composited-child.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/repaint/fixed-pos-with-composited-child.html
diff --git a/LayoutTests/compositing/repaint/fixed-pos-with-composited-child.html b/LayoutTests/compositing/repaint/fixed-pos-with-composited-child.html
index d59ecd571377034e71b040df496141ffb5a0771e..9326e11c66849f29e1c06db0df8c529e52d4ec92 100644
--- a/LayoutTests/compositing/repaint/fixed-pos-with-composited-child.html
+++ b/LayoutTests/compositing/repaint/fixed-pos-with-composited-child.html
@@ -2,6 +2,7 @@
<html>
<head>
+<script src="../../resources/run-after-display.js"></script>
<style>
.fixedContainer {
@@ -23,8 +24,10 @@
</style>
<script>
- if (window.testRunner)
+ if (window.testRunner) {
testRunner.dumpAsText();
+ testRunner.waitUntilDone();
+ }
if (window.internals) {
/* Note carefully, compositing for fixed position is _disabled_ here */
@@ -34,15 +37,18 @@
function test()
{
- testRunner.display();
+ runAfterDisplay(function() {
+ if (window.internals)
+ window.internals.startTrackingRepaints(document);
- if (window.internals)
- window.internals.startTrackingRepaints(document);
+ window.scrollTo(0, 100);
- window.scrollTo(0, 100);
+ if (window.internals)
+ document.getElementById('layers').textContent = window.internals.layerTreeAsText(document, internals.LAYER_TREE_INCLUDES_REPAINT_RECTS);
- if (window.internals)
- document.getElementById('layers').textContent = window.internals.layerTreeAsText(document, internals.LAYER_TREE_INCLUDES_REPAINT_RECTS);
+ if (window.testRunner)
+ testRunner.notifyDone();
+ });
}
</script>
</head>

Powered by Google App Engine
This is Rietveld 408576698