Bug 40584

Summary:

[Qt] fast/text/justify-padding-distribution.html fails

Product:

WebKit

Reporter:

Csaba Osztrogonác <ossy>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

webkit.review.bot

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Test passes. It should be removed from skipped list.	none

Csaba Osztrogonác

Reported 2010-06-14 12:45:50 PDT

fast/text/justify-padding-distribution.html introduced in http://trac.webkit.org/changeset/61128 . It fails on "Qt Linux Release" bot. pretty diff: http://build.webkit.org/results/Qt%20Linux%20Release/r61135%20%2813292%29/fast/text/justify-padding-distribution-pretty-diff.html The test has this style rule: div.test { font-family: ahem; text-align: justify; width: 290px; font-size: 10px; } But rendertree dump contains 9px height text: - RenderInline {SPAN} at (0,0) size 60x10 - RenderText {#text} at (0,10) size 60x10 + RenderInline {SPAN} at (0,0) size 60x9 + RenderText {#text} at (0,10) size 60x9

Attachments
Test passes. It should be removed from skipped list. (1.20 KB, patch) 2011-10-20 07:10 PDT, Zoltan Arvai	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Csaba Osztrogonác

Comment 1 2010-06-14 13:07:33 PDT

Zoltan Arvai

Comment 2 2011-10-20 07:10:11 PDT

Created attachment 111765 [details] Test passes. It should be removed from skipped list. Test passes: fast/text/justify-padding-distribution.html

Andreas Kling

Comment 3 2011-10-20 07:14:09 PDT

Comment on attachment 111765 [details] Test passes. It should be removed from skipped list. rs=me

WebKit Review Bot

Comment 4 2011-10-20 08:17:49 PDT

Comment on attachment 111765 [details] Test passes. It should be removed from skipped list. Clearing flags on attachment: 111765 Committed r97980: <http://trac.webkit.org/changeset/97980>

WebKit Review Bot

Comment 5 2011-10-20 08:17:53 PDT

All reviewed patches have been landed. Closing bug.

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