LineBuilder does not align runs vertically (actually no alignment at all)
Created attachment 405801 [details] Patch
<rdar://problem/66736366>
Created attachment 407394 [details] Patch
Comment on attachment 407394 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407394&action=review > Source/WebCore/layout/inlineformatting/InlineLineBuilder.cpp:178 > - // and the trimmed width should be capped at std::min(run.trailingWhitespaceWidth(), overflowWidth) for texgt runs. Both FF and Chrome agree. > + // and the trimmed width should be capped at std::min(run.trailingWhitespaceWidth(), overflowWidth) for text runs. Both FF and Chrome agree. Great spelling fixes! > Source/WebCore/layout/inlineformatting/InlineLineBuilder.cpp:394 > - // Note that just becasue the trimmable width is 0 (font-size: 0px), it does not mean we don't have a trimmable trailing content. > + // Note that just because the trimmable width is 0 (font-size: 0px), it does not mean we don't have a trimmable trailing content. Here too. > Source/WebCore/layout/inlineformatting/InlineLineBuilder.cpp:401 > - // Do not add trimmable letter spacing after a fully trimmable whitesapce. > + // Do not add trimmable letter spacing after a fully trimmable whitespace. And here.
Created attachment 407402 [details] Patch
Created attachment 407403 [details] Patch
Created attachment 407432 [details] Patch
Created attachment 407438 [details] Patch
Created attachment 407444 [details] Patch
Committed r266271: <https://trac.webkit.org/changeset/266271> All reviewed patches have been landed. Closing bug and clearing flags on attachment 407444 [details].