Bug 11064

Summary:

manual-tests/bidi-parens.html is invalid

Product:

WebKit

Reporter:

mitz

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED CONFIGURATION CHANGED

Severity:

Minor

CC:

ahmad.saleem792, ap, darin, koivisto, mmaxfield, rniwa, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
Safari 15.5 matches other browsers	none

Description mitz 2006-09-27 13:01:55 PDT

The CSS selector in manual-tests/bidi-parens.html is wrong so the font-family rule isn't applied to the Hebrew text. The test is also misleading because it doesn't mention that it requires a third-party font that typical OS X installations don't have.

Comment 1 Ahmad Saleem 2022-06-20 04:52:40 PDT

Created attachment 460353 [details]
Safari 15.5 matches other browsers

Based on commit from Webkit Github mirror - https://github.com/WebKit/WebKit/commit/ea30f082a7a9aa52c9e4fe39543813f57c0f6113

This is added in 2015 and I can see, it is present in SVN trunk as well:

https://svn.webkit.org/repository/webkit/trunk/ManualTests/

Search for "bidi".

In the attached screenshot, you can see latest output across all browsers and how they reflect. I am not sure based on Comment 01, whether it is something different expected here but I do see difference in font rendering across all.

Comment 2 Radar WebKit Bug Importer 2022-06-20 04:53:08 PDT

<rdar://problem/95530544>

Comment 3 Ryosuke Niwa 2022-06-20 09:01:32 PDT

The test looks functional now. Perhaps we should convert this to a ref test & make it automatic.

Comment 4 mitz 2022-06-20 09:44:49 PDT

Probably not much value in doing that.

Comment 5 Ryosuke Niwa 2022-06-20 23:01:04 PDT

OK, let's just close this as config changed then.