RESOLVED FIXED 167820
Simple line layout: Bail out from Simple Line Layout when the primary font is insufficient. 
https://bugs.webkit.org/show_bug.cgi?id=167820
Summary Simple line layout: Bail out from Simple Line Layout when the primary font is...
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.