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

Unified Diff: LayoutTests/compositing/squashing/resize-squashing-layer-that-needs-full-repaint.html

Issue 331693004: Issue repaints when offsetFromRenderer changes for any squashed layer. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed tests. 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
Index: LayoutTests/compositing/squashing/resize-squashing-layer-that-needs-full-repaint.html
diff --git a/LayoutTests/compositing/squashing/resize-squashing-layer-that-needs-full-repaint.html b/LayoutTests/compositing/squashing/resize-squashing-layer-that-needs-full-repaint.html
new file mode 100644
index 0000000000000000000000000000000000000000..b5953dbd86ba21b462c0b2436f26495a349b8b4a
--- /dev/null
+++ b/LayoutTests/compositing/squashing/resize-squashing-layer-that-needs-full-repaint.html
@@ -0,0 +1,37 @@
+<!doctype html>
+<script src="../../resources/run-after-display.js"></script>
+
+<!-- The second two divs form a combined squashing layer. display:none'ing the first of them will resize the squashing layer. -->
+<div style="width: 500px; height: 500px; -webkit-transform: translateZ(0)"></div>
+<div style="position: absolute; top: 55px; left: 55px; width: 500px; height: 500px; background-color: lightblue"></div>
+<div id="to-be-removed" style="position: absolute; top: 0px; left: 0px; width: 50px; height: 50px; background-color: lightgray"></div>
+
+<pre id="output" style="display: none"></pre>
+<script>
+// This test checks that resizing a squashing layer such that the offset of content squashed into it relative to the squashing
+// container causes a repaint of its bounds within the updated squashing layer geometry.
+if (window.testRunner) {
+ testRunner.dumpAsText();
+ testRunner.waitUntilDone();
+}
+
+if (window.internals)
+ internals.settings.setLayerSquashingEnabled(true);
+
+setTimeout(function() {
+ if (window.internals)
+ window.internals.startTrackingRepaints(document);
+ document.querySelector("#to-be-removed").style.display = 'none';
+
+ var output = document.querySelector("#output");
+ if (window.internals) {
+ output.textContent = window.internals.layerTreeAsText(document, internals.LAYER_TREE_INCLUDES_REPAINT_RECTS);
+ window.internals.stopTrackingRepaints(document);
+ }
+ output.style.display = 'block';
+
+ if (window.testRunner)
+ testRunner.notifyDone();
+
+}, 2000);
Ian Vollick 2014/06/13 02:42:48 Oof, that's a very big timeout. Why's this necessa
chrishtr 2014/06/13 04:24:24 Ugh, that's me being sloppy. I changed from runAft
+</script>

Powered by Google App Engine
This is Rietveld 408576698