it really is a builder
<rdar://problem/57474964>
Created attachment 384302 [details] Patch
Comment on attachment 384302 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384302&action=review > Source/WebCore/ChangeLog:3 > + [LFC][IFC] Rename Line to LineBuilder What an excellent name! > Source/WebCore/layout/inlineformatting/LineLayoutContext.cpp:162 > +LineLayoutContext::IsEndOfLine LineLayoutContext::placeInlineItem(LineBuilder& line, const InlineItem& inlineItem) Maybe the variables like this should be renamed too? Maybe in another patch...
(In reply to Antti Koivisto from comment #3) > Comment on attachment 384302 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=384302&action=review > > > Source/WebCore/ChangeLog:3 > > + [LFC][IFC] Rename Line to LineBuilder > > What an excellent name! > > > Source/WebCore/layout/inlineformatting/LineLayoutContext.cpp:162 > > +LineLayoutContext::IsEndOfLine LineLayoutContext::placeInlineItem(LineBuilder& line, const InlineItem& inlineItem) > > Maybe the variables like this should be renamed too? Maybe in another > patch... Yeah, I was planning on doing it in a separate patch.
Comment on attachment 384302 [details] Patch Clearing flags on attachment: 384302 Committed r252865: <https://trac.webkit.org/changeset/252865>
All reviewed patches have been landed. Closing bug.