There are a number of platform checks that are implemented as version checks in this file.
Created attachment 388751 [details] Patch
Comment on attachment 388751 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388751&action=review > Source/WTF/wtf/PlatformHave.h:571 > +#define HAVE_FONT_STYLE_0 1 > +#define HAVE_FONT_STYLE_4 1 I think "font style 0" is simply too brief to be unambiguous. I think it should be more like SYSTEM_FONT_STYLE_TITLE_0. > Source/WebCore/rendering/RenderThemeIOS.mm:1269 > +#if !PLATFORM(WATCHOS) && !PLATFORM(APPLETV) Forgot to use HAVE here. > Source/WebCore/rendering/RenderThemeIOS.mm:1287 > +#if !PLATFORM(WATCHOS) && !PLATFORM(APPLETV) And here.
Created attachment 388857 [details] Patch
Comment on attachment 388857 [details] Patch Clearing flags on attachment: 388857 Committed r255219: <https://trac.webkit.org/changeset/255219>
All reviewed patches have been landed. Closing bug.
<rdar://problem/58942377>