<rdar://74884402>
Created attachment 432721 [details] Patch
Created attachment 432722 [details] Patch
Created attachment 432761 [details] Patch
Created attachment 432812 [details] Patch
Created attachment 432987 [details] Patch
Created attachment 432989 [details] Patch
Created attachment 432997 [details] Patch
Created attachment 433006 [details] Patch
Created attachment 433024 [details] Patch
Comment on attachment 433024 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433024&action=review > Source/WebCore/rendering/LegacyInlineFlowBox.cpp:659 > + // FIXME: Let's do device pixel snapping at paint time instead. Can you reference a bugzilla bug in the comment? The "Let's do" makes it hard to judge how important this is; it should be more like "we should not do integral rounding here". > Source/WebCore/rendering/LegacyInlineFlowBox.cpp:771 > if (isRootBox) { Sucks how there's if (isRootBox) logic way up at the start of this function, and here. Would be nice to factor the middle part into separate function.
Created attachment 433050 [details] Patch
Committed r279673 (239480@main): <https://commits.webkit.org/239480@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 433050 [details].