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

Unified Diff: third_party/WebKit/LayoutTests/paint/inline-block-overflow-repaint.html

Issue 2896363003: Ensure visual overflow from style recalc gets propagated (Closed)
Patch Set: bug 724453 Created 3 years, 7 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: third_party/WebKit/LayoutTests/paint/inline-block-overflow-repaint.html
diff --git a/third_party/WebKit/LayoutTests/paint/inline-block-overflow-repaint.html b/third_party/WebKit/LayoutTests/paint/inline-block-overflow-repaint.html
new file mode 100644
index 0000000000000000000000000000000000000000..a73cfee1038cc63cf90db0fb38caa5fc18d0d578
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/paint/inline-block-overflow-repaint.html
@@ -0,0 +1,16 @@
+<!DOCTYPE html>
Xianzhu 2017/05/23 20:35:42 Please put this file under paint/invalidation dire
rhogan 2017/05/23 21:32:16 Done.
+<style>
+p {overflow:hidden;}
Xianzhu 2017/05/23 20:35:42 I suggest removing the above line, because whether
rhogan 2017/05/23 21:32:16 Mm, not with you here - we do need the container t
Xianzhu 2017/05/23 21:59:47 I found I misunderstood the bug. I thought it were
+#test1 {overflow:hidden; display: inline-block;}
+#test2 {overflow:hidden; display: block;}
+</style>
+<p> crbug.com/724453: There should be only an outline on the right-hand side and in the descent below the text.</p>
+<p><a href="" id="test1"><span>Text</span></a></p>
+<p> crbug.com/724453: There should be no outline below.</p>
+<p><a href="" id="test2"><span>Text</span></a></p>
+<script>
+document.body.offsetTop;
Xianzhu 2017/05/23 20:35:42 Please use <script src="../../resources/run-after-
rhogan 2017/05/23 21:32:16 Done.
+document.getElementById("test1").style['outline'] = 'solid black 10px';
+document.getElementById("test2").style['outline'] = 'solid black 10px';
+</script>
+

Powered by Google App Engine
This is Rietveld 408576698