RESOLVED FIXED 97188
[GTK] Layout Test fast/text/word-space-with-kerning.html is failing 
https://bugs.webkit.org/show_bug.cgi?id=97188
Summary [GTK] Layout Test fast/text/word-space-with-kerning.html is failing
Zan Dobersek
Reported 2012-09-20 03:11:20 PDT
The following layout test is failing on GTK fast/text/word-space-with-kerning.html The test was introduced in r129068 http://trac.webkit.org/changeset/129068
Attachments
Add attachment proposed patch, testcase, etc.
Martin Robinson
Comment 1 2012-12-11 05:29:46 PST
This test appears to be passing now, perhaps with the switch to Harfbuzz for complex text rendering. I'll unskip it.
Dominik Röttsches (drott)
Comment 2 2012-12-11 07:14:57 PST
Really? I thought this would be blocked by bug 102603, at least the test name suggests that.
Martin Robinson
Comment 3 2012-12-11 07:21:15 PST
(In reply to comment #2) > Really? I thought this would be blocked by bug 102603, at least the test name suggests that. Interesting. The bots were reporting this test as passing. Perhaps we are inadvertently passing?
Dominik Röttsches (drott)
Comment 4 2012-12-12 08:53:09 PST
(In reply to comment #3) > Interesting. The bots were reporting this test as passing. Perhaps we are inadvertently passing? So okay, I think it's correct that this passes now, but I am not sure it had something to do with switching to Harfbuzz. From the introduction to this test case (r129068): " (WebCore::RenderText::computePreferredLogicalWidths): When kerning is enabled, words are measured with their trailing space, if there is one, then the width of a space is subtracted. Changed that width, stored in the local variable wordTrailingSpaceWidth, to include the word spacing, since it is included in the result of widthFromCache()." So this failed only when kerning was enabled. Since the -webkit-font-kerning: normal; in this test currently does not have any effect with the FreeType backend, the test passes.
Note You need to log in before you can comment on or make changes to this bug.