Bug 106015 - [Qt] New tests introduced in r138632 fails
: [Qt] New tests introduced in r138632 fails
Status: RESOLVED INVALID
: WebKit
New Bugs
: 528+ (Nightly build)
: Unspecified Unspecified
: P2 Normal
Assigned To:
:
: Qt, QtTriaged
:
: 87008 92591
  Show dependency treegraph
 
Reported: 2013-01-03 06:26 PST by
Modified: 2014-02-03 03:24 PST (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2013-01-03 06:26:36 PST
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
------- Comment #1 From 2013-01-03 06:29:54 PST -------
I skipped them to paint the bots green - r138708.
Please don't forget to unskip them with the proper fix.
------- Comment #2 From 2013-01-04 07:15:33 PST -------
Only fail on GTK, Qt and EFL. Could this be subpixel-related?
------- Comment #3 From 2013-02-11 03:27:57 PST -------
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.)
------- Comment #4 From 2013-02-18 00:34:13 PST -------
Those tests are now passing on EFL port. We recently enabled subpixel layout so this may indeed be related to it.
------- Comment #5 From 2014-02-03 03:24:07 PST -------
=== 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.