ssia
Created attachment 413533 [details] Patch
Created attachment 413536 [details] Patch
Comment on attachment 413536 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413536&action=review > Source/WebCore/layout/inlineformatting/InlineLineGeometry.h:38 > + InlineLineGeometry(const InlineRect& lineLogicalRect, const InlineLayoutSize& lineBoxLogicalSize, InlineLayoutUnit aligmentBaseline, InlineLayoutUnit horizontalAlignmentOffset); Ok as-is. Consider removing names for first 2 params now that they have different types.
Comment on attachment 413536 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413536&action=review > Source/WebCore/layout/integration/LayoutIntegrationPagination.cpp:134 > moveVertically(line.rect(), offset), > + line.rect().size(), > moveVertically(line.rect(), offset), Not from this patch but the last one should be enclosingRect().
Created attachment 413542 [details] Patch
Committed r269571: <https://trac.webkit.org/changeset/269571> All reviewed patches have been landed. Closing bug and clearing flags on attachment 413542 [details].
<rdar://problem/71168177>
(In reply to Daniel Bates from comment #3) > Comment on attachment 413536 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=413536&action=review > > > Source/WebCore/layout/inlineformatting/InlineLineGeometry.h:38 > > + InlineLineGeometry(const InlineRect& lineLogicalRect, const InlineLayoutSize& lineBoxLogicalSize, InlineLayoutUnit aligmentBaseline, InlineLayoutUnit horizontalAlignmentOffset); > > Ok as-is. Consider removing names for first 2 params now that they have > different types. I'd rather keep them as one is about the line while the other is about the line box geometry.