Bug 88963 – [Qt] New css3/flexbox/flexitem.html fails

• Component: Tools / Tests
• Status: RESOLVED INVALID

[Csaba Osztrogonác]

css3/flexbox/flexitem.html introduced in http://trac.webkit.org/changeset/120132 and fails on Qt:

--- /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/css3/flexbox/flexitem-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-qt5/buildslave/qt-linux-32-release-qt5/build/layout-test-results/css3/flexbox/flexitem-actual.txt 
@@ -2,7 +2,24 @@
 
 object
 
-PASS
+FAIL:
+Expected 100 for width, but got 95. 
+Expected 100 for width, but got 95. 
+Expected 100 for width, but got 95. 
+Expected 100 for width, but got 95. 
+Expected 100 for width, but got 123. 
+Expected 100 for width, but got 95. 
+
+<div class="flexbox">
+  <button data-expected-display="block" data-expected-width="100">button</button>
+  <canvas data-expected-display="block" data-expected-width="100">canvas</canvas>
+  <iframe data-expected-display="block" data-expected-width="100" src="data:text/html,&lt;body bgcolor=#fff&gt;iframe&lt;/body&gt;"></iframe>
+  <object data-expected-display="block" data-expected-width="100">object</object>
+  <select data-expected-display="block" data-expected-width="100">
+    <option>select</option>
+  </select>
+  <textarea data-expected-display="block" data-expected-width="100">textarea</textarea>
+</div>
 
 PASS
 
@@ -17,7 +34,7 @@
 
 FAIL:
 Expected 100 for height, but got 20. 
-Expected 100 for height, but got 20. 
+Expected 100 for height, but got 21. 
 
 <div class="flexbox">
   <img data-expected-display="block" data-expected-width="200" style="-webkit-flex: 1 0 auto;" src="../images/resources/blue-100.png">

[Csaba Osztrogonác]

I skipped it to paint the bot green - r120163. Please unskip it with the proper fix.

[Tony Chang]

I investigated this failure.  This is because in WebKitQt, the padding on <select>s (MenuLists) are hard coded.  This happens in Source/WebKit/qt/WebCoreSupport/RenderThemeQStyle.cpp in RenderThemeQStyle::setPopupPadding.

The test tries to set the padding to 0, but it doesn't stick.

I think the bug is that the user can't set padding on <select>s.

The second failure must be the alt text being taller than 20px. I can fix that.

[Tony Chang]

Oh, if it's intentional that setting padding on a <select> does nothing, then we could just rebaseline for Qt.

[Tony Chang]

Created attachment 147386 [details]
Patch

[Tony Chang]

Committed r120330: <http://trac.webkit.org/changeset/120330>

[Tony Chang]

(In reply to comment #5)
> Committed r120330: <http://trac.webkit.org/changeset/120330>

@@ -17,7 +34,7 @@

 FAIL:
 Expected 100 for height, but got 20. 
-Expected 100 for height, but got 20. 
+Expected 100 for height, but got 21.

Should now be fixed.  I need guidance from Qt on whether it's a bug that you can't set padding on <select>s or if we should check in a Qt specific baseline.

[Jocelyn Turcotte]

=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.