Bug 110381

Summary: Line layout (but not pref widths) double-counts word spacing when between inlines
Product: WebKit Reporter: Levi Weintraub <leviw>
Component: Layout and RenderingAssignee: Levi Weintraub <leviw>
Status: RESOLVED FIXED    
Severity: Normal CC: eae, eric, esprehn+autocc, mitz, ojan.autocc, robert, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: https://code.google.com/p/chromium/issues/detail?id=170226
Attachments:
Description Flags
Patch
none
Patch none

Levi Weintraub
Reported 2013-02-20 14:14:26 PST
We correctly determine the preferred logical width for a line with word-spacing set and spaces between inlines, but when laying out the line, we add word spacing an extra time which leads to wrapping inline-blocks. Example: http://codepen.io/jeremychurch/pen/BFLbi
Attachments
Patch (4.57 KB, patch)
2013-02-20 14:27 PST, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
Patch (5.27 KB, patch)
2013-02-20 14:47 PST, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Levi Weintraub
Comment 1 2013-02-20 14:27:36 PST
Created attachment 189380 [details] Patch
Emil A Eklund
Comment 2 2013-02-20 14:32:01 PST
Comment on attachment 189380 [details] Patch Is the bug limited to floats or does it also apply to inline-block elements? If so could you please add a line of inline-blocks to the test?
Levi Weintraub
Comment 3 2013-02-20 14:47:11 PST
Created attachment 189383 [details] Patch
WebKit Review Bot
Comment 4 2013-02-20 15:46:04 PST
Comment on attachment 189383 [details] Patch Clearing flags on attachment: 189383 Committed r143520: <http://trac.webkit.org/changeset/143520>
WebKit Review Bot
Comment 5 2013-02-20 15:46:08 PST
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.