Bug 3693 - Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN
Summary: Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Dave Hyatt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-24 01:54 PDT by mitz
Modified: 2005-11-04 00:31 PST (History)
0 users

See Also:

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+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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