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

Unified Diff: LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html

Issue 324733002: Add a layout test for iframe flicker due to overflow scroll promotion (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: . Created 6 years, 6 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
« no previous file with comments | « no previous file | LayoutTests/compositing/iframes/composited-iframe-transition-flicker-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html
diff --git a/LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html b/LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html
new file mode 100644
index 0000000000000000000000000000000000000000..903281454e1e6e67728a7f7f62646090950e5aee
--- /dev/null
+++ b/LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html
@@ -0,0 +1,35 @@
+<!DOCTYPE HTML>
+<iframe id="frame" srcdoc="<style>input { width: 100px; }</style><input type='text' id='input' value='add stuff here'></input>"></iframe>
ojan 2014/06/09 17:42:37 Nit: type=text is the default.
ojan 2014/06/09 17:42:38 Nit: type=text is the default.
Ian Vollick 2014/06/09 18:35:56 Done.
+<script>
+if (window.testRunner) {
+ testRunner.dumpAsText();
ojan 2014/06/09 17:42:38 Nit: 4 space indent?
Ian Vollick 2014/06/09 18:35:56 Done.
+ testRunner.waitUntilDone();
+}
+
+onload = function() {
+ var layerTreePreStyleUpdate = "";
+ var layerTreePostStyleUpdate = "";
+ if (window.internals)
+ window.internals.settings.setAcceleratedCompositingForOverflowScrollEnabled(true);
+
+ requestAnimationFrame(function() {
+ document.getElementById("frame").contentDocument.getElementById("input").value =
+ "This is a very long string that will cause overflow";
+ layerTreePreStyleUpdate = window.internals.layerTreeAsText(document);
+
+ // Force a style recalc.
+ document.body.offsetTop;
+ layerTreePostStyleUpdate = window.internals.layerTreeAsText(document);
+
+ requestAnimationFrame(function() {
ojan 2014/06/09 17:42:37 I don't think we need the requestAnimationFrame si
Ian Vollick 2014/06/09 18:35:56 Done.
+ var pre = document.createElement('pre');
+ if (layerTreePreStyleUpdate === layerTreePostStyleUpdate)
+ pre.innerHTML = "PASS - layer trees are the same.";
+ else
+ pre.innerHTML = "FAIL - layer trees are not the same.";
+ document.body.appendChild(pre);
+ testRunner.notifyDone();
+ });
+ });
+};
+</script>
« no previous file with comments | « no previous file | LayoutTests/compositing/iframes/composited-iframe-transition-flicker-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698