Update css3/flexbox/auto-margins.html after r177774
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jan 2015 23:56:01 +0000 (23:56 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jan 2015 23:56:01 +0000 (23:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=139975

Patch by Myles C. Maxfield <litherum@gmail.com> on 2015-01-07
Reviewed by Darin Adler.

This is a subpixel problem. Flexbox uses SimpleLineLayout for the entire
box and therefore does all its math in floating point, while the
-expected uses the regular renderer hierarchy and therefore fixed-point
math. This patch updates the test to use Ahem so that the character
widths divide the same way in both cases.

* css3/flexbox/auto-margins-expected.html:
* css3/flexbox/auto-margins.html:
* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@178063 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/css3/flexbox/auto-margins-expected.html
LayoutTests/css3/flexbox/auto-margins.html
LayoutTests/platform/mac/TestExpectations

index 4507731f6e8a91fc3344154f88f097affb195ed1..cda1dd44c5ba4ddbaedfcea005b87fb98aa22169 100644 (file)
@@ -1,3 +1,20 @@
+2015-01-07  Myles C. Maxfield  <litherum@gmail.com>
+
+        Update css3/flexbox/auto-margins.html after r177774
+        https://bugs.webkit.org/show_bug.cgi?id=139975
+
+        Reviewed by Darin Adler.
+
+        This is a subpixel problem. Flexbox uses SimpleLineLayout for the entire
+        box and therefore does all its math in floating point, while the
+        -expected uses the regular renderer hierarchy and therefore fixed-point
+        math. This patch updates the test to use Ahem so that the character
+        widths divide the same way in both cases.
+
+        * css3/flexbox/auto-margins-expected.html:
+        * css3/flexbox/auto-margins.html:
+        * platform/mac/TestExpectations:
+
 2015-01-07  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Unreviewed test gardening (writing-mode).
 2015-01-07  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Unreviewed test gardening (writing-mode).
index 66d52c3c51270cb5a021420dd7fc1cd76dd8db32..0cc382ef4bbe1e4461a53b5b660f1bc4ab55dfed 100644 (file)
 <body>
 <p>These tests are from the spec: <a href="http://dev.w3.org/csswg/css3-flexbox/#auto-margins">http://dev.w3.org/csswg/css3-flexbox/#auto-margins</a>.</p>
 
 <body>
 <p>These tests are from the spec: <a href="http://dev.w3.org/csswg/css3-flexbox/#auto-margins">http://dev.w3.org/csswg/css3-flexbox/#auto-margins</a>.</p>
 
-<p>The word OK should be centered vertically and horizontally.</p>
+<p>A black box should be centered vertically and horizontally.</p>
 <div style="width: 4em; height: 4em; background: silver">
 <div style="width: 4em; height: 4em; background: silver">
-  <table style="width: 100%; height: 100%;"><tr><td style="text-align: center">OK</td></tr></table>
+  <table style="width: 100%; height: 100%;"><tr><td style="text-align: center; font-family: Ahem;">OK</td></tr></table>
 </div>
 
 <div style="width: 4em; height: 4em; margin-top: 10px; background: silver; -webkit-writing-mode: vertical-rl">
 </div>
 
 <div style="width: 4em; height: 4em; margin-top: 10px; background: silver; -webkit-writing-mode: vertical-rl">
-  <table style="width: 100%; height: 100%;"><tr><td style="text-align: center">OK</td></tr></table>
+  <table style="width: 100%; height: 100%;"><tr><td style="text-align: center; font-family: Ahem;">OK</td></tr></table>
 </div>
 
 <p>You should see 3 blue concentric circles.</p>
 </div>
 
 <p>You should see 3 blue concentric circles.</p>
index b63e3ca05c4d4b685918f51ee07e56e9863d8e7c..edf580e9e1bb2a841dcd1811c580551c3c89d744 100644 (file)
 <body>
 <p>These tests are from the spec: <a href="http://dev.w3.org/csswg/css3-flexbox/#auto-margins">http://dev.w3.org/csswg/css3-flexbox/#auto-margins</a>.</p>
 
 <body>
 <p>These tests are from the spec: <a href="http://dev.w3.org/csswg/css3-flexbox/#auto-margins">http://dev.w3.org/csswg/css3-flexbox/#auto-margins</a>.</p>
 
-<p>The word OK should be centered vertically and horizontally.</p>
+<p>A black box should be centered vertically and horizontally.</p>
 <div class="flexbox" style="width: 4em; height: 4em; background: silver">
 <div class="flexbox" style="width: 4em; height: 4em; background: silver">
-  <p id="ok" style="margin: auto;">OK</p>
+  <p id="ok" style="margin: auto; font-family: Ahem;">OK</p>
 </div>
 
 <div class="flexbox" style="width: 4em; height: 4em; margin-top: 10px; background: silver; -webkit-writing-mode: vertical-rl">
 </div>
 
 <div class="flexbox" style="width: 4em; height: 4em; margin-top: 10px; background: silver; -webkit-writing-mode: vertical-rl">
-  <p id="okVertical" style="margin: auto;">OK</p>
+  <p id="okVertical" style="margin: auto; font-family: Ahem;">OK</p>
 </div>
 
 <p>You should see 3 blue concentric circles.</p>
 </div>
 
 <p>You should see 3 blue concentric circles.</p>
index 3f500233f23f49af46c83be67f94f4e8a6e13963..541f7b803fe6e3a4537802cc984b9a16919719d4 100644 (file)
@@ -1339,7 +1339,6 @@ fast/canvas/canvas-fillRect-gradient-shadow.html [ Pass Failure ]
 
 # Kerning, Ligatures, and Printer Fonts caused these tests to fail
 # The following tests are reftests
 
 # Kerning, Ligatures, and Printer Fonts caused these tests to fail
 # The following tests are reftests
-webkit.org/b/139968 css3/flexbox/auto-margins.html [ ImageOnlyFailure ]
 webkit.org/b/139968 css3/flexbox/csswg/flexbox_direction-row-reverse.html [ ImageOnlyFailure ]
 webkit.org/b/139968 fast/box-decoration-break/box-decoration-break-rendering.html [ ImageOnlyFailure ]
 webkit.org/b/139968 fast/css-generated-content/after-with-inline-continuation.html [ ImageOnlyFailure ]
 webkit.org/b/139968 css3/flexbox/csswg/flexbox_direction-row-reverse.html [ ImageOnlyFailure ]
 webkit.org/b/139968 fast/box-decoration-break/box-decoration-break-rendering.html [ ImageOnlyFailure ]
 webkit.org/b/139968 fast/css-generated-content/after-with-inline-continuation.html [ ImageOnlyFailure ]