Bug 67772 - [Qt] fast/css/line-after-floating-div.html fails
Summary: [Qt] fast/css/line-after-floating-div.html fails
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 67725
  Show dependency treegraph
 
Reported: 2011-09-08 02:34 PDT by Oliver Varga
Modified: 2014-02-03 03:18 PST (History)
4 users (show)

See Also:


Attachments
Add to skipped. (1.04 KB, patch)
2011-09-08 02:42 PDT, Oliver Varga
ossy: review-
ossy: commit-queue-
Details | Formatted Diff | Diff
failing image (6.00 KB, image/png)
2011-09-08 03:34 PDT, Csaba Osztrogonác
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Varga 2011-09-08 02:34:44 PDT
The pixel test result is bad.
Comment 1 Oliver Varga 2011-09-08 02:42:38 PDT
Created attachment 106716 [details]
Add to skipped.
Comment 2 Csaba Osztrogonác 2011-09-08 02:49:15 PDT
Comment on attachment 106716 [details]
Add to skipped.

View in context: https://bugs.webkit.org/attachment.cgi?id=106716&action=review

> LayoutTests/ChangeLog:4
> +        Gardening: add fast/css/line-after-floating-div.html test to skipped.
> +        https://bugs.webkit.org/show_bug.cgi?id=67772

[Qt] fast/css/line-after-floating-div.html fails
https://bugs.webkit.org/show_bug.cgi?id=67772

> LayoutTests/ChangeLog:8
> +        * platform/qt/Skipped:

* platform/qt/Skipped: Skip it until fix.

> LayoutTests/platform/qt/Skipped:2414
> +# [Qt] fast/css/line-after-floating-div.html fail on Qt added in r94710.

# new test introduced in r94710, but fails on Qt.
Comment 3 Csaba Osztrogonác 2011-09-08 02:51:04 PDT
(In reply to comment #2)
> # new test introduced in r94710, but fails on Qt.

And this test didn't introduced in r94710.
Comment 4 Csaba Osztrogonác 2011-09-08 02:51:39 PDT
r94696
Comment 5 Csaba Osztrogonác 2011-09-08 03:16:55 PDT
Skipped by http://trac.webkit.org/changeset/94751, leave open to fix the real bug.
Comment 7 Ryosuke Niwa 2011-09-08 09:24:05 PDT
(In reply to comment #6)
> Created an attachment (id=106719) [details]
> failing image

Is this what you're seeing on Qt?  Then it's passing!  The test passes if you don't see red regions as it says so on the test.
Comment 8 Csaba Osztrogonác 2011-09-08 12:19:56 PDT
But at the bottom of the square there is a one pixel width red line.
Comment 9 Ryosuke Niwa 2011-09-08 12:22:51 PDT
(In reply to comment #8)
> But at the bottom of the square there is a one pixel width red line.

Mn... that's interesting.  It's probably rounding error though so I would worry too much about it.  It used to have 12px or so red region.
Comment 10 Ádám Kallai 2012-10-11 02:18:18 PDT
Is Anybody working this bug? This test still fails on Qt.
Comment 11 Jocelyn Turcotte 2014-02-03 03:18:53 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.