407227 - bogus min-width calculation with images separated by whitespace

Status: RESOLVED FIXED

(Core :: Layout: Block and Inline, defect, P2)

Description

[Robert O'Callahan (:roc) (email my personal email if necessary)]

See testcase. The table cell should break after the space, but it doesn't. This testcase was made by Chris Pearce.

Comment 1

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: testcase

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: fix

AddInlineMinWidthForFlow needs to do an OptionallyBreak when there's a break opportunity right at the end of the nsLineBreaker-textrun (i.e. where it's interrupted by some non-text content).

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 291978 [details] [diff] [review]
fix

r+sr=dbaron

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 291978 [details] [diff] [review]
fix

fairly simple fix for fairly major layout regression

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

checked in