Bug 147862

Summary: [iOS] Arabic letter Yeh is drawn in LastResort
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, jonlee, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch darin: review+

Myles C. Maxfield
Reported 2015-08-10 19:18:50 PDT
[iOS] Arabic letter Yeh is drawn in LastResort
Attachments
Patch (9.29 KB, patch)
2015-08-10 19:36 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (9.97 KB, patch)
2015-08-10 19:39 PDT, Myles C. Maxfield 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-08-10 19:36:11 PDT
Created attachment 258687 [details] Patch
Myles C. Maxfield
Comment 2 2015-08-10 19:36:36 PDT
<rdar://problem/22202935>
Myles C. Maxfield
Comment 3 2015-08-10 19:39:20 PDT
Created attachment 258689 [details] Patch
Simon Fraser (smfr)
Comment 4 2015-08-10 20:42:04 PDT
Comment on attachment 258689 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=258689&action=review > Source/WebCore/ChangeLog:33 > + Test: fast/text/arabic-glyph-cache-fill-combine.html If this problem is so general, why is it that the test is iOS-specific? Can you construct a Mac test?
Myles C. Maxfield
Comment 5 2015-08-11 08:38:57 PDT
Comment on attachment 258689 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=258689&action=review >> Source/WebCore/ChangeLog:33 >> + Test: fast/text/arabic-glyph-cache-fill-combine.html > > If this problem is so general, why is it that the test is iOS-specific? Can you construct a Mac test? The bug requires 1. Successive Unicode code points which combine to form a grapheme cluster, 2. Such a combining sequence must not occur across a boundary of 256 code points, and 3. A font which supports all the constituents of the combined grapheme cluster. I'll see if I can find such a case.
Myles C. Maxfield
Comment 6 2015-08-11 11:06:01 PDT
Committed r188263: <http://trac.webkit.org/changeset/188263>
Note You need to log in before you can comment on or make changes to this bug.