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

Unified Diff: LayoutTests/fast/block/basic/fieldset-with-legend-right-margin.html

Issue 1173853005: LayoutFieldSet::computePreferredLogicalWidths uses marginLeft for marginRight by mistake Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Adding layout Test Created 5 years, 5 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/fast/block/basic/fieldset-with-legend-right-margin-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/block/basic/fieldset-with-legend-right-margin.html
diff --git a/LayoutTests/fast/block/basic/fieldset-with-legend-right-margin.html b/LayoutTests/fast/block/basic/fieldset-with-legend-right-margin.html
new file mode 100644
index 0000000000000000000000000000000000000000..7d43e8183d5d623ce491fff61f341a79035023c1
--- /dev/null
+++ b/LayoutTests/fast/block/basic/fieldset-with-legend-right-margin.html
@@ -0,0 +1,32 @@
+<style>
esprehn 2015/07/13 06:02:42 Add <!DOCTYPE html>
+ legend { height: 10px; border: 1px solid blue; }
+ fieldset { margin-bottom: 10px;border: 2px solid green; }
+ div.rel { position: relative; height: 100px;border:2px solid red; }
esprehn 2015/07/13 06:02:42 missing space after ; in a couple places
+</style>
+<script src="../../../resources/check-layout.js"></script>
+<body onload="checkLayout('fieldset');">
+ <div class="rel" style="width: 500px;">
+ <fieldset data-expected-margin-right="10" style="position: absolute;margin-left:20px;margin-right:10px;">
a.suchit 2015/07/13 12:59:09 we are correcting the legend margin right so data-
+ <legend style="width: 150px;margin-left:10px;margin-right:20px"></legend>
+ <h1 style="background-color:green;margin:5px">HEADING 1</h1>
+ </fieldset>
+ </div>
+ <div class="rel" style="width: 500px;">
+ <fieldset data-expected-margin-right="20" style="position: absolute;margin-left:40px;margin-right:20px;">
+ <legend style="width: 150px;margin-left:10px;margin-right:20px"></legend>
+ <h1 style="background-color:green;margin:5px">HEADING 1</h1>
+ </fieldset>
+ </div>
+ <div class="rel" style="width: 500px;margin-left :20px;">
+ <fieldset data-expected-margin-right="-20" style="position: absolute;margin-left:-20px;margin-right:-20px;">
+ <legend style="width: 150px;margin-left:10px;margin-right:20px"></legend>
+ <h1 style="background-color:green;margin:5px">HEADING 1</h1>
+ </fieldset>
+ </div>
+ <div class="rel" style="width: 500px;">
+ <fieldset data-expected-margin-right="-10" style="position: absolute;width:500px;margin-left:20px;margin-right:-10px;">
+ <legend style="width: 150px;margin-left:10px;margin-right:20px"></legend>
+ <h1 style="background-color:green;margin:5px">HEADING 1</h1>
+ </fieldset>
+ </div>
+</body>
« no previous file with comments | « no previous file | LayoutTests/fast/block/basic/fieldset-with-legend-right-margin-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698