I'm going to fix WEBCORE_TESTSUPPORT_EXPORT mis-use.
Created attachment 382882 [details] Patch
Comment on attachment 382882 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382882&action=review Makes sense and looks like EWS is happy with it. > Source/WebCore/ChangeLog:3 > + [Win] WEBCORE_TESTSUPPORT_EXPORT should be used only for WebCoreTestSupport, but for WebCore Nit: "not" for WebCore
Comment on attachment 382882 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382882&action=review >> Source/WebCore/ChangeLog:3 >> + [Win] WEBCORE_TESTSUPPORT_EXPORT should be used only for WebCoreTestSupport, but for WebCore > > Nit: "not" for WebCore Will fix. > Source/WebKit/config.h:-40 > -#define WEBCORE_TESTSUPPORT_EXPORT WTF_EXPORT_DECLARATION I should remove WEBCORE_TESTSUPPORT_EXPORT definition in webcore not only for windows, but also for linux and mac in order to avoid build breakage only for windows.
Created attachment 382996 [details] Patch
Comment on attachment 382882 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382882&action=review >> Source/WebKit/config.h:-40 >> -#define WEBCORE_TESTSUPPORT_EXPORT WTF_EXPORT_DECLARATION > > I should remove WEBCORE_TESTSUPPORT_EXPORT definition in webcore not only for windows, but also for linux and mac in order to avoid build breakage only for windows. Umm, it's not easy to do that. WebCore and WebCoreTestSupport shares WebCore/config.h which includes WebCore/platform/PlatformExportMacros.h. I will do it if Windows-only build breakage will happen frequently.
Comment on attachment 382996 [details] Patch Clearing flags on attachment: 382996 Committed r252166: <https://trac.webkit.org/changeset/252166>
All reviewed patches have been landed. Closing bug.
<rdar://problem/56968155>