Bug 44942 - REGRESSION (r61921): RTL text in <b> tag doesn't display in WebKit under certain conditions
Summary: REGRESSION (r61921): RTL text in <b> tag doesn't display in WebKit under cert...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://www.bneybraki.com/120090/%D7%9...
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2010-08-31 04:34 PDT by Jeremy Moskovich
Modified: 2011-02-02 20:14 PST (History)
3 users (show)

See Also:


Attachments
Screenshot (258.51 KB, image/jpeg)
2010-08-31 04:36 PDT, Jeremy Moskovich
no flags Details
Reduced test case (1011 bytes, text/html)
2010-08-31 04:37 PDT, Jeremy Moskovich
no flags Details
testcase reduced even further - minimal, I think. (1.30 KB, text/html)
2010-08-31 05:03 PDT, Aharon (Vladimir) Lanin
no flags Details
Still smaller (just less text). (953 bytes, text/html)
2010-08-31 05:06 PDT, Aharon (Vladimir) Lanin
no flags Details
Do not bail out if the explicit embedding sequence was a no-op (20.51 KB, patch)
2011-02-02 20:03 PST, mitz
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Moskovich 2010-08-31 04:34:20 PDT
One of the lines in the URL above (see attached screenshot) isn't displayed in WebKit, but shows up fine in FF & IE.

Reduced testcase attached.

Marking as a regression due to the fact this doesn't reproduce on Safari 5.0 on Windows, still need to test with older versions of Safari on Mac to be sure.

Browsers tested:
Safari 5.0 Windows: OK
Safari 5.0.1/Mac : FAIL
WebKit nightly (r66356): FAIL
Comment 1 Jeremy Moskovich 2010-08-31 04:36:12 PDT
Created attachment 66032 [details]
Screenshot
Comment 2 Jeremy Moskovich 2010-08-31 04:37:14 PDT
Created attachment 66033 [details]
Reduced test case
Comment 3 Aharon (Vladimir) Lanin 2010-08-31 05:03:31 PDT
Created attachment 66036 [details]
testcase reduced even further - minimal, I think.

The <b> is not actually relevant - a <span> will do just as nicely. The table stuff and the RTL are necessary, however. A further-reduced test case attached, see comments inside.
Comment 4 Aharon (Vladimir) Lanin 2010-08-31 05:06:18 PDT
Created attachment 66037 [details]
Still smaller (just less text).
Comment 5 mitz 2010-08-31 21:42:39 PDT
Regressed in <http://trac.webkit.org/changeset/61921>.
Comment 6 mitz 2010-08-31 21:43:28 PDT
<rdar://problem/8380506>
Comment 7 mitz 2011-02-02 20:03:42 PST
Created attachment 81025 [details]
Do not bail out if the explicit embedding sequence was a no-op
Comment 8 mitz 2011-02-02 20:11:48 PST
Comment on attachment 81025 [details]
Do not bail out if the explicit embedding sequence was a no-op

Sam gave this an enthusiastic r+
Comment 9 mitz 2011-02-02 20:14:49 PST
Committed r77461.
http://trac.webkit.org/changeset/77461