Bug 148261

Summary: [iOS] Emoji's descent is way too large
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED INVALID    
Severity: Normal CC: dino, enrica, jonlee, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch simon.fraser: review+

Myles C. Maxfield
Reported 2015-08-20 16:08:05 PDT
Emoji's descent is way too large
Attachments
Patch (8.68 KB, patch)
2015-08-20 16:18 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (8.69 KB, patch)
2015-08-20 17:43 PDT, Myles C. Maxfield 		simon.fraser: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-08-20 16:18:45 PDT
Created attachment 259515 [details] Patch
Myles C. Maxfield
Comment 2 2015-08-20 16:21:08 PDT
Comment on attachment 259515 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=259515&action=review > Source/WebCore/platform/graphics/cocoa/FontCocoa.mm:-275 > - if (m_platformData.isEmoji()) { This should match the size clamping code too
Myles C. Maxfield
Comment 3 2015-08-20 17:43:38 PDT
Created attachment 259541 [details] Patch
Myles C. Maxfield
Comment 4 2015-08-20 17:45:36 PDT
<rdar://problem/22194708>
Myles C. Maxfield
Comment 5 2015-08-20 18:26:15 PDT
This is no longer necessary because of https://bugs.webkit.org/show_bug.cgi?id=148265
Myles C. Maxfield
Comment 6 2015-08-20 19:13:52 PDT
Fixed by r188737
Note You need to log in before you can comment on or make changes to this bug.