Bug 3693

Summary: Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Dave Hyatt <hyatt>
Status: CLOSED FIXED    
Severity: Normal    
Priority: P2    
Version: 412   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Testcase
none
Proposed fix darin: review+

mitz
Reported 2005-06-24 01:54:35 PDT
Layout order of the testcase is incorrect in WebKit. Firefox renders correctly. This is http://bugs.kde.org/show_bug.cgi?id=72528 which has been fixed there: http://websvn.kde.org/ trunk/kdelibs/khtml/rendering/bidi.cpp?rev=368932&r1=367649&r2=368932 The KHTML patch is applicable. The testcase can serve as a layout test.
Attachments
Testcase (322 bytes, text/html)
2005-06-24 01:56 PDT, mitz 		no flags
Details
Proposed fix (691 bytes, patch)
2005-06-24 01:59 PDT, mitz 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2005-06-24 01:56:19 PDT
Created attachment 2624 [details] Testcase Based on http://bugs.kde.org/attachment.cgi?id=4395&action=view Can be used as layout test.
mitz
Comment 2 2005-06-24 01:59:18 PDT
Created attachment 2625 [details] Proposed fix Sane as the KHTML patch http://websvn.kde.org/trunk/kdelibs/khtml/rendering/bidi.cpp?rev=368932&r1=367649&r2=368932 Fixes the testcase, doesn't break any of the current layout tests.
mitz
Comment 3 2005-06-24 02:04:37 PDT
(In reply to comment #2) > Sane Oops, I meant "same".
Darin Adler
Comment 4 2005-06-24 08:31:36 PDT
Comment on attachment 2625 [details] Proposed fix r=me
Note You need to log in before you can comment on or make changes to this bug.