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

Unified Diff: LayoutTests/compositing/force-compositing-mode/overflow-hidden-iframe-layer.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/force-compositing-mode/overflow-hidden-iframe-layer.html
diff --git a/LayoutTests/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html b/LayoutTests/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html
index ddd4b84f0730a4ca06545a705f1826baebfbb9f5..9950451e9d92175f15fda63c2e1af3ad7b9a3b9e 100644
--- a/LayoutTests/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html
+++ b/LayoutTests/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html
@@ -1,5 +1,6 @@
<html>
<head>
+ <script src="../../resources/run-after-display.js"></script>
<style>
iframe {
height: 150px;
@@ -13,15 +14,19 @@
}
function doTest()
{
- if (window.testRunner)
- testRunner.display();
- var iframe = document.getElementById("parent-iframe");
- iframe.contentDocument.body.setAttribute("style", "overflow:hidden");
- iframe.contentDocument.body.offsetWidth; // See bug 41999
if (window.testRunner) {
- testRunner.dumpAsText();
- document.getElementById('layertree').innerText = window.internals.layerTreeAsText(document);
+ testRunner.waitUntilDone();
}
+ runAfterDisplay(function() {
+ var iframe = document.getElementById("parent-iframe");
+ iframe.contentDocument.body.setAttribute("style", "overflow:hidden");
+ iframe.contentDocument.body.offsetWidth; // See bug 41999
+ if (window.testRunner) {
+ testRunner.dumpAsText();
+ document.getElementById('layertree').innerText = window.internals.layerTreeAsText(document);
+ testRunner.notifyDone();
+ }
+ });
}
window.addEventListener("load", doTest, false);
</script>

Powered by Google App Engine
This is Rietveld 408576698