Bug 106015 – [Qt] New tests introduced in r138632 fails

• Status: RESOLVED
• Resolution: INVALID
• Component: New Bugs

[Csaba Osztrogonác]

There are 2 new tests introduced in https://trac.webkit.org/changeset/138632,
but they fail on Qt with the following diffs:

--- /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/fast/css-generated-content/pseudo-animation-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/fast/css-generated-content/pseudo-animation-actual.txt 
@@ -4,10 +4,10 @@
 
 
 PASS div.offsetWidth is 52
-PASS div.offsetWidth is 20
+FAIL div.offsetWidth should be 20. Was 19.
 PASS div.offsetWidth is 12
 PASS div.offsetWidth is 52
-PASS div.offsetWidth is 20
+FAIL div.offsetWidth should be 20. Was 19.
 PASS div.offsetWidth is 12
 PASS successfullyParsed is true
 


--- /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/fast/css-generated-content/pseudo-transition-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/fast/css-generated-content/pseudo-transition-actual.txt 
@@ -4,10 +4,10 @@
 
 
 PASS div.offsetWidth is 52
-PASS div.offsetWidth is 20
+FAIL div.offsetWidth should be 20. Was 19.
 PASS div.offsetWidth is 12
 PASS div.offsetWidth is 52
-PASS div.offsetWidth is 20
+FAIL div.offsetWidth should be 20. Was 19.
 PASS div.offsetWidth is 12
 PASS successfullyParsed is true

[Csaba Osztrogonác]

I skipped them to paint the bots green - r138708.
Please don't forget to unskip them with the proper fix.

[Zan Dobersek]

Only fail on GTK, Qt and EFL. Could this be subpixel-related?

[Zan Dobersek]

These two tests now pass on GTK and had their failure expectations removed. It might be OK for Qt and EFL to unskip them.
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&showAllRuns=true&tests=fast%2Fcss-generated-content%2Fpseudo-animation.html%2Cfast%2Fcss-generated-content%2Fpseudo-transition.html
(Note that the debug builder data is outdated.)

[Chris Dumez]

Those tests are now passing on EFL port. We recently enabled subpixel layout so this may indeed be related to it.

[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.