Bug 44942

Summary: REGRESSION (r61921): RTL text in <b> tag doesn't display in WebKit under certain conditions
Product: WebKit Reporter: Jeremy Moskovich <playmobil>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: jshin, mitz, xji
Priority: P2 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: OS X 10.5   
URL: http://www.bneybraki.com/120090/%D7%9E%D7%94-%D7%9C%D7%A2%D7%A9%D7%95%D7%AA-%D7%A2%D7%9D-%D7%94%D7%9B%D7%A1%D7%A3
Attachments:
Description Flags
Screenshot
none
Reduced test case
none
testcase reduced even further - minimal, I think.
none
Still smaller (just less text).
none
Do not bail out if the explicit embedding sequence was a no-op mitz: review+

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