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

Unified Diff: LayoutTests/fast/repaint/padding-keeping-visual-size.html

Issue 340323002: Don't force full invalidation on padding or margin change (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Split tests; One new test 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/fast/repaint/padding-keeping-visual-size.html
diff --git a/LayoutTests/fast/repaint/padding-keeping-visual-size.html b/LayoutTests/fast/repaint/padding-keeping-visual-size.html
new file mode 100644
index 0000000000000000000000000000000000000000..54e28ce154c9e11e3d03ca29883320abc19fe903
--- /dev/null
+++ b/LayoutTests/fast/repaint/padding-keeping-visual-size.html
@@ -0,0 +1,40 @@
+<!DOCTYPE HTML>
+<script src="resources/text-based-repaint.js"></script>
+<script>
+function repaintTest()
+{
+ // No visual change. Optimally, there should be no repaint.
+ var target1 = document.getElementById('target1');
+ target1.style.padding = '20px';
+ target1.style.width = '60px';
+ target1.style.height = '60px';
+
+ // Content will shrink. Should repaint at least the content.
+ var target2 = document.getElementById('target2');
+ target2.style.padding = '20px';
+ target2.style.width = '60px';
+ target2.style.height = '60px';
+}
+window.onload = runRepaintTest;
+</script>
+<style>
+body {
+ margin: 0;
+}
+.outer {
+ width: 100px;
+ height: 100px;
+ position: absolute;
+ background-color: green;
+}
+#target2 {
+ top: 200px;
+}
+.inner {
+ width: 100%;
+ height: 100%;
+ background-color: yellow;
+}
+</style>
+<div id="target1" class="outer"></div>
+<div id="target2" class="outer"><div class="inner"></div></div>

Powered by Google App Engine
This is Rietveld 408576698