The ENABLE() macro is used before "config.h" gets a chance to be included, which makes the compilation trip. In the usual arches (at least for the ones tested by EWS bots) unified builds are masking the issue.
Created attachment 388236 [details] Patch
Comment on attachment 388236 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388236&action=review > Source/WebCore/layout/integration/LayoutIntegrationBoxTree.cpp:30 > -#if ENABLE(LAYOUT_FORMATTING_CONTEXT) > - > #include "config.h" > + > +#if ENABLE(LAYOUT_FORMATTING_CONTEXT) > + > #include "LayoutIntegrationBoxTree.h" The header is already guarded, so this should be: #include "config.h" #include "LayoutIntegrationBoxTree.h" #if ENABLE(LAYOUT_FORMATTING_CONTEXT)
Created attachment 388237 [details] Patch for landing
The commit-queue encountered the following flaky tests while processing attachment 388237 [details]: editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org) The commit-queue is continuing to process your patch.
Comment on attachment 388237 [details] Patch for landing Clearing flags on attachment: 388237 Committed r254825: <https://trac.webkit.org/changeset/254825>
All reviewed patches have been landed. Closing bug.
<rdar://problem/58738949>