Bug 14758

Summary: REGRESSION: Repeated text after line break on facebook.com
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Critical Keywords: HasReduction, InRadar, Regression
Priority: P1    
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.facebook.com/whatsnew.php
Attachments:
Description Flags
Reduction
none
Reduction
none
Re-add two lines that were removed for absolutely no reason in r24485 darin: review+

Description mitz 2007-07-24 14:09:31 PDT
Several lines on the page in the URL the are supposed to wrap just before a link are duplicated after the break, with the link tacked on the same line as the duplicate. Going to attach a reduction and try to find out if it was the bidi algorithm patch or the break-all/break-word patch that broke this (I am pretty sure it was one of them).
Comment 1 mitz 2007-07-24 14:10:30 PDT
Created attachment 15671 [details]
Reduction
Comment 2 mitz 2007-07-24 14:18:14 PDT
Created attachment 15672 [details]
Reduction

Smaller reduction
Comment 3 mitz 2007-07-24 15:10:50 PDT
Got a fix. Running tests now.
Comment 4 mitz 2007-07-24 15:38:36 PDT
Created attachment 15673 [details]
Re-add two lines that were removed for absolutely no reason in r24485

No layout test regressions. Includes a new layout test to ensure even I don't do it again.
Comment 5 Darin Adler 2007-07-24 15:39:32 PDT
Comment on attachment 15673 [details]
Re-add two lines that were removed for absolutely no reason in r24485

r=me
Comment 6 David Kilzer (:ddkilzer) 2007-07-24 21:27:48 PDT
<rdar://problem/5359088>
Comment 7 Darin Adler 2007-07-25 08:53:32 PDT
Committed revision 24621.