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

Unified Diff: LayoutTests/compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-composite.html

Issue 196653012: Remove display() from compositing/ LayoutTests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: softwarecompositing expectations Created 6 years, 9 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-overflow-div-scrolled-late-composite.html
diff --git a/LayoutTests/compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-composite.html b/LayoutTests/compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-composite.html
index 1f082895f1c4e55c6fa9ababbd2716f0f007e02f..1cfb470305e15eb714c1da6b03f8529976f794d2 100644
--- a/LayoutTests/compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-composite.html
+++ b/LayoutTests/compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-composite.html
@@ -17,6 +17,7 @@
<div style="position: relative; left: 10px; top: 80px">
This test is successful if "Target Link" above is covered in a green rectangle with rounded corners.
</div>
+<script src="../../resources/run-after-display.js"></script>
<script>
function runTest() {
var targetDiv = document.getElementById('targetDiv');
@@ -32,16 +33,15 @@ function runTest() {
// then scrolls that div, then exits. If everything is working correctly, the link highlight will still
// be displayed correctly after all these operations.
if (window.eventSender && window.testRunner) {
-
eventSender.gestureShowPress(x, y);
- window.testRunner.display();
-
- document.getElementById('divToForceCompositedLayer').style.webkitTransform = 'translateZ(0)';
- document.body.offsetTop;
- window.testRunner.display();
-
- targetDiv.scrollTop += 20;
- window.setTimeout(function() { window.testRunner.notifyDone(); }, 0);
+ runAfterDisplay(function() {
+ document.getElementById('divToForceCompositedLayer').style.webkitTransform = 'translateZ(0)';
+ document.body.offsetTop;
+ runAfterDisplay(function() {
+ targetDiv.scrollTop += 20;
+ testRunner.notifyDone();
+ });
+ });
} else {
debug("This test requires DumpRenderTree.");
}

Powered by Google App Engine
This is Rietveld 408576698