Bug 97280

Summary:

REGRESSION (r126763): Incorrect line breaking when both kerning and word spacing are enabled

Product:

WebKit

Reporter:

mitz

Component:

Text

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ned

Priority:

Keywords:

Regression

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

URL:

data:text/html,%3Cdiv%20style=%22border:%20solid;%20width:%2050px;%20word-spacing:%2050px;%20-webkit-font-kerning:%20normal;%22%3Ea%20b%3C/div%3E

Attachments:

Description	Flags
Make TextLayout::width() apply word spacing except at the first character of a run	adele: review+

Description mitz 2012-09-20 18:48:38 PDT

To reproduce:
run-webkit-tests --complex fast/text/basic/004.html

Line breaking isn’t accounting for word spacing correctly.

This was caused by <http://trac.webkit.org/r126763>, the fix for bug 83045.

Comment 1 mitz 2012-09-20 21:01:39 PDT

Please ignore the original description. The failure happens when the test is modified to enable kerning.

Comment 2 mitz 2012-09-20 21:08:46 PDT

The URL contains a reduction.
<div style="border: solid; width: 50px; word-spacing: 50px; -webkit-font-kerning: normal;">a b</div>

Comment 3 mitz 2012-09-20 23:41:31 PDT

TextLayout uses fontWithNoWordSpacing…

Comment 4 mitz 2012-09-21 00:21:21 PDT

Created attachment 165064 [details]
Make TextLayout::width() apply word spacing except at the first character of a run

Comment 5 mitz 2012-09-21 09:21:48 PDT

Fixed in <http://trac.webkit.org/r129233>.