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

Unified Diff: LayoutTests/compositing/force-compositing-mode/overflow-iframe-leave-compositing.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-iframe-leave-compositing.html
diff --git a/LayoutTests/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html b/LayoutTests/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html
index e8790b682dc03c3a5b878b44635d2df280aa50ba..0bff6844c5cba8fd3185208ada64902aa5613cd8 100644
--- a/LayoutTests/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html
+++ b/LayoutTests/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html
@@ -1,5 +1,6 @@
<html>
<head>
+ <script src="../../resources/run-after-display.js"></script>
<style>
iframe {
height: 150px;
@@ -14,15 +15,18 @@
function doTest()
{
if (window.testRunner)
- testRunner.display();
- var iframe = document.getElementById("parent-iframe");
- var box = iframe.contentDocument.getElementById("box");
- box.setAttribute("style", "height:50px;width:50px;");
- 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");
+ var box = iframe.contentDocument.getElementById("box");
+ box.setAttribute("style", "height:50px;width:50px;");
+ 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