Bug 167820

Summary: Simple line layout: Bail out from Simple Line Layout when the primary font is insufficient.
Product: WebKit Reporter: alan <zalan>
Component: Layout and RenderingAssignee: alan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, esprehn+autocc, glenn, koivisto, kondapallykalyan, mmaxfield, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

alan
Reported 2017-02-03 16:29:05 PST
Currently SSL only works with the primary font.
Attachments
Patch (8.96 KB, patch)
2017-02-03 16:45 PST, alan 		no flags
DetailsFormatted DiffDiff
Patch (9.00 KB, patch)
2017-02-03 17:27 PST, alan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2017-02-03 16:45:25 PST
Created attachment 300579 [details] Patch
Myles C. Maxfield
Comment 2 2017-02-03 16:47:30 PST
(In reply to comment #0) > Currently SSL only works with the primary font. I think you mean SLL. Pretty sure SSL works with all fonts. 😂
Radar WebKit Bug Importer
Comment 3 2017-02-03 16:47:31 PST
<rdar://problem/30359685>
Myles C. Maxfield
Comment 4 2017-02-03 16:50:39 PST
Comment on attachment 300579 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300579&action=review > Source/WebCore/rendering/SimpleLineLayout.cpp:172 > + auto glyphData = fontCascade.glyphDataForCharacter(character, false); reference please
alan
Comment 5 2017-02-03 17:27:54 PST
Created attachment 300582 [details] Patch
WebKit Commit Bot
Comment 6 2017-02-03 17:43:40 PST
Comment on attachment 300582 [details] Patch Clearing flags on attachment: 300582 Committed r211661: <http://trac.webkit.org/changeset/211661>
WebKit Commit Bot
Comment 7 2017-02-03 17:43:48 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.