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+

Description mitz 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.
Comment 1 mitz 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.
Comment 2 mitz 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.
Comment 3 mitz 2005-06-24 02:04:37 PDT 
(In reply to comment #2)

> Sane

Oops, I meant "same".
Comment 4 Darin Adler 2005-06-24 08:31:36 PDT 
Comment on attachment 2625 [details]
Proposed fix

r=me