Bug 167827

Summary: Simple line layout: Remove redundant codepath check.
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <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   
Bug Depends on: 167855    
Bug Blocks:    
Attachments:
Description Flags
Patch none

zalan
Reported 2017-02-03 20:42:10 PST
RenderText already has this information.
Attachments
Patch (2.96 KB, patch)
2017-02-03 20:49 PST, zalan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2017-02-03 20:42:33 PST
<rdar://problem/30361850>
zalan
Comment 2 2017-02-03 20:49:44 PST
Created attachment 300597 [details] Patch
Myles C. Maxfield
Comment 3 2017-02-03 21:27:22 PST
Comment on attachment 300597 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300597&action=review > Source/WebCore/rendering/SimpleLineLayout.cpp:-109 > - FlowFontIsNotSimple = 1LLU << 43, Wait, what about all the uses of this symbol? Don't they have to be updated?
WebKit Commit Bot
Comment 4 2017-02-03 22:04:57 PST
Comment on attachment 300597 [details] Patch Clearing flags on attachment: 300597 Committed r211668: <http://trac.webkit.org/changeset/211668>
WebKit Commit Bot
Comment 5 2017-02-03 22:05:02 PST
All reviewed patches have been landed. Closing bug.
WebKit Commit Bot
Comment 6 2017-02-05 09:34:30 PST
Re-opened since this is blocked by bug 167855
Note You need to log in before you can comment on or make changes to this bug.