Bug 203876 - WEBCORE_TESTSUPPORT_EXPORT should be used only for WebCoreTestSupport, not for WebCore
Summary: WEBCORE_TESTSUPPORT_EXPORT should be used only for WebCoreTestSupport, not fo...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-11-05 18:46 PST by Fujii Hironori
Modified: 2019-11-06 18:30 PST (History)
18 users (show)

See Also:

Attachments
Patch (11.46 KB, patch)
2019-11-05 18:56 PST, Fujii Hironori 		no flags Details | Formatted Diff | Diff
Patch (11.45 KB, patch)
2019-11-06 18:17 PST, Fujii Hironori 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2019-11-05 18:46:08 PST 
I'm going to fix WEBCORE_TESTSUPPORT_EXPORT mis-use.
Comment 1 Fujii Hironori 2019-11-05 18:56:20 PST 
Created attachment 382882 [details]
Patch
Comment 2 Ross Kirsling 2019-11-06 12:18:50 PST 
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 3 Fujii Hironori 2019-11-06 14:50:18 PST 
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.
Comment 4 Fujii Hironori 2019-11-06 18:17:01 PST 
Created attachment 382996 [details]
Patch
Comment 5 Fujii Hironori 2019-11-06 18:20:09 PST 
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 6 Fujii Hironori 2019-11-06 18:29:52 PST 
Comment on attachment 382996 [details]
Patch

Clearing flags on attachment: 382996

Committed r252166: <https://trac.webkit.org/changeset/252166>
Comment 7 Fujii Hironori 2019-11-06 18:29:56 PST 
All reviewed patches have been landed.  Closing bug.
Comment 8 Radar WebKit Bug Importer 2019-11-06 18:30:17 PST 
<rdar://problem/56968155>