Bug 34176 - Certain markup causes RTL errors and test displays out of order
Summary: Certain markup causes RTL errors and test displays out of order
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Normal
Assignee: Nobody
URL:
Keywords:
: 52497 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-01-26 12:02 PST by Tim McConnell
Modified: 2011-02-01 12:30 PST (History)
4 users (show)

See Also:

Attachments
Markup that causes RTL display problems. (706 bytes, text/html)
2010-01-26 12:02 PST, Tim McConnell 		no flags Details
patch w/ layout test (141.54 KB, patch)
2011-01-25 16:39 PST, Xiaomei Ji 		hyatt: 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 Tim McConnell 2010-01-26 12:02:13 PST 
Created attachment 47431 [details]
Markup that causes RTL display problems.

Within an RTL block, some content will display incorrectly. For example:

<div>This is a <span style="text-decoration: underline;"><em>Test<span></span></em></span></div>

In this case, "Test" appears before the rest of the content. It occurs when the spans and styles become complex enough.
Comment 1 Jeremy Moskovich 2010-10-19 08:22:34 PDT 
Tested on OS X 10.6:

FF 3.6.6: OK
Opera 10.63: OK
Safari 5.0.1: FAIL
Comment 2 Xiaomei Ji 2011-01-25 16:39:34 PST 
Created attachment 80146 [details]
patch w/ layout test
Comment 3 Xiaomei Ji 2011-01-25 16:43:42 PST 
*** Bug 52497 has been marked as a duplicate of this bug. ***
Comment 4 Dave Hyatt 2011-02-01 10:29:18 PST 
Comment on attachment 80146 [details]
patch w/ layout test

r=me
Comment 5 Xiaomei Ji 2011-02-01 12:02:36 PST 
Committed r77267: <http://trac.webkit.org/changeset/77267>
Comment 6 Dimitri Glazkov (Google) 2011-02-01 12:30:49 PST 
(In reply to comment #5)
> Committed r77267: <http://trac.webkit.org/changeset/77267>

The layout test has both executable bit and a wrong mime-type set. Can you please fix?

Also, please add Chromium baselines when they are ready on the bots.