Bug 67772 – [Qt] fast/css/line-after-floating-div.html fails

• Status: RESOLVED
• Resolution: INVALID
• Component: Tools / Tests

[Oliver Varga]

The pixel test result is bad.

[Oliver Varga]

Created attachment 106716 [details]
Add to skipped.

[Csaba Osztrogonác]

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.

[Csaba Osztrogonác]

(In reply to comment #2)
> # new test introduced in r94710, but fails on Qt.

And this test didn't introduced in r94710.

[Csaba Osztrogonác]

r94696

[Csaba Osztrogonác]

Skipped by http://trac.webkit.org/changeset/94751, leave open to fix the real bug.

[Csaba Osztrogonác]

Created attachment 106719 [details]
failing image

Mac result: http://trac.webkit.org/browser/trunk/LayoutTests/platform/mac/fast/css/line-after-floating-div-expected.png

[Ryosuke Niwa]

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

[Csaba Osztrogonác]

But at the bottom of the square there is a one pixel width red line.

[Ryosuke Niwa]

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

[Ádám Kallai]

Is Anybody working this bug? This test still fails on Qt.

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