Bug 178424

Summary: REGRESSION (r223476): WebCore exports symbols with names belonging to other frameworks
Product: WebKit Reporter: mitz
Component: WebCore Misc.Assignee: Keith Miller <keith_miller>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, buildbot, cdumez, cmarcelo, commit-queue, dbates, ddkilzer, keith_miller, thorton, webkit-bug-importer
Priority: P1 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=178229
Attachments:
Description Flags
Patch none

Description mitz 2017-10-17 20:19:22 PDT 
<https://trac.webkit.org/r223476>, the fix for bug 178229, made WebCore export symbols identical to the ones it’s “soft linking” using the SOFT_LINK_FUNCTION_FOR_SOURCE macro. For example, now WebCore exports a function called CMClockGetTime, which is different from the CMClockGetTime that the Core Media framework exports. WebCore’s exports should all be in WebCore’s C++ namespaces or have the Web or _Web prefix.
Comment 1 mitz 2017-10-17 20:27:00 PDT 
<rdar://problem/35045237>
Comment 2 Keith Miller 2017-10-18 19:43:07 PDT 
*** Bug 178495 has been marked as a duplicate of this bug. ***
Comment 3 Keith Miller 2017-10-18 19:44:28 PDT 
Created attachment 324202 [details]
Patch
Comment 4 Keith Miller 2017-10-18 19:47:42 PDT 
From nm WebCore.build/Objects-normal/x86_64/XSLTUnicodeSort.o (which calls a function, xsltEvalAttrValueTemplate, from libxslt):

(__TEXT,__text) weak private external _xsltEvalAttrValueTemplate
Comment 5 David Kilzer (:ddkilzer) 2017-10-19 09:52:22 PDT 
Comment on attachment 324202 [details]
Patch

r=me.  We should stop soft-linking libxslt as well, but that's a separate patch.
Comment 6 WebKit Commit Bot 2017-10-19 11:18:35 PDT 
Comment on attachment 324202 [details]
Patch

Clearing flags on attachment: 324202

Committed r223693: <https://trac.webkit.org/changeset/223693>
Comment 7 WebKit Commit Bot 2017-10-19 11:18:37 PDT 
All reviewed patches have been landed.  Closing bug.