Bug 106015 - [Qt] New tests introduced in r138632 fails
Summary: [Qt] New tests introduced in r138632 fails
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 87008 92591
  Show dependency treegraph
 
Reported: 2013-01-03 06:26 PST by Csaba Osztrogonác
Modified: 2014-02-03 03:24 PST (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 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 Csaba Osztrogonác 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 Zan Dobersek 2013-01-04 07:15:33 PST
Only fail on GTK, Qt and EFL. Could this be subpixel-related?
Comment 3 Zan Dobersek 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 Chris Dumez 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 Jocelyn Turcotte 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.