not enabled yet.
Created attachment 450983 [details] Patch
Comment on attachment 450983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=450983&action=review > Source/WebCore/layout/formattingContexts/inline/InlineLineBoxBuilder.cpp:207 > + InlineLayoutUnit ascent = fontMetrics.ascent(useIdeographicBaseline ? IdeographicBaseline : AlphabeticBaseline); > + InlineLayoutUnit descent = fontMetrics.descent(useIdeographicBaseline ? IdeographicBaseline : AlphabeticBaseline); Maybe this code could just use the existing FontBaseline enum everywhere (though it should be enum class)? > Source/WebCore/layout/formattingContexts/inline/InlineLineBoxBuilder.h:52 > + enum class UseIdeographicBaseline : uint8_t { No, Yes }; Replacing this too.
good point
Created attachment 451000 [details] Patch
Committed r289183 (246875@main): <https://commits.webkit.org/246875@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 451000 [details].
<rdar://problem/88548584>