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

Unified Diff: LayoutTests/compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.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/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.html
diff --git a/LayoutTests/compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.html b/LayoutTests/compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.html
index 8fa58e5b8b6a677671341ec73a5ec35697580b2c..5d08ecaafac7821efd395f50759c0f077a23402c 100644
--- a/LayoutTests/compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.html
+++ b/LayoutTests/compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-composite.html
@@ -2,6 +2,7 @@
<html>
<head>
<script src="../../resources/js-test.js"></script>
+<script src="../../resources/run-after-display.js"></script>
</head>
<body onLoad="runTest();">
<iframe id="targetFrame" src="resources/1-frame-noncomposited.html" style="position: relative; left: 10px; top: 10px; width: 400px; height: 150px;"></iframe>
@@ -22,13 +23,13 @@ function runTest() {
if (window.eventSender && window.testRunner) {
eventSender.gestureShowPress(x, y);
- window.testRunner.display();
-
- document.getElementById("targetFrame").contentDocument.getElementById('divToControlCompositing').style.webkitTransform = "translateZ(0)";
- window.testRunner.display();
-
- document.getElementById("targetFrame").contentWindow.scrollTo(0, 20);
- window.setTimeout(function() { window.testRunner.notifyDone(); }, 0);
+ runAfterDisplay(function() {
+ document.getElementById("targetFrame").contentDocument.getElementById('divToControlCompositing').style.webkitTransform = "translateZ(0)";
+ runAfterDisplay(function() {
+ document.getElementById("targetFrame").contentWindow.scrollTo(0, 20);
+ testRunner.notifyDone();
+ });
+ });
} else {
debug("This test requires DumpRenderTree.");
}

Powered by Google App Engine
This is Rietveld 408576698