r251171 breaks the internal macOS Catalina build.
It would seem that <HIServices/AccessibilityPriv.h> is exposed via the ApplicationServices framework. Though I might be missing something here (I probably am :/)
Created attachment 381052 [details] Patch
Comment on attachment 381052 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381052&action=review > Source/WebKit/Platform/spi/mac/ApplicationServicesSPI.h:33 > +#pragma clang diagnostic ignored "-Wundef" Is this still needed? The location ApplicationServicesPriv.h is being imported from should be recognized as a system header location and therefore exempt from warnings. > Source/WebKit/Platform/spi/mac/ApplicationServicesSPI.h:59 > +extern AXClientType _AXGetClientForCurrentRequestUntrusted(void); I know that this is just being moved around, but normally we keep function declarations outside the #if/#else/#endif since it’s harmless to repeat a declaration and it gives us extra protection in case the declaration in the private header changes.
Comment on attachment 381052 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381052&action=review Thanks for the review! >> Source/WebKit/Platform/spi/mac/ApplicationServicesSPI.h:33 >> +#pragma clang diagnostic ignored "-Wundef" > > Is this still needed? The location ApplicationServicesPriv.h is being imported from should be recognized as a system header location and therefore exempt from warnings. Fixed! >> Source/WebKit/Platform/spi/mac/ApplicationServicesSPI.h:59 >> +extern AXClientType _AXGetClientForCurrentRequestUntrusted(void); > > I know that this is just being moved around, but normally we keep function declarations outside the #if/#else/#endif since it’s harmless to repeat a declaration and it gives us extra protection in case the declaration in the private header changes. Sounds good — moved out of the #if/#else/#endif.
Created attachment 381053 [details] For EWS
Comment on attachment 381053 [details] For EWS Clearing flags on attachment: 381053 Committed r251179: <https://trac.webkit.org/changeset/251179>
All reviewed patches have been landed. Closing bug.
<rdar://problem/56320543>