Bug 221625

Summary: [GTK] Export additional methods of SourceBufferPrivate to support hidden visibility
Product: WebKit Reporter: Don Olmstead <don.olmstead>
Component: PlatformAssignee: Don Olmstead <don.olmstead>
Status: RESOLVED FIXED    
Severity: Normal CC: mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Don Olmstead 2021-02-09 13:41:51 PST
....
Comment 1 Don Olmstead 2021-02-09 13:48:37 PST
Created attachment 419757 [details]
Patch
Comment 2 Michael Catanzaro 2021-02-09 13:52:34 PST
I can only imagine why these are needed, especially since WebCore is the toplevel library... I guess a test program is using them?
Comment 3 Don Olmstead 2021-02-09 14:38:19 PST
(In reply to Michael Catanzaro from comment #2)
> I can only imagine why these are needed, especially since WebCore is the
> toplevel library... I guess a test program is using them?

From https://ews-build.webkit.org/#/builders/36/builds/27419 which didn't export those symbols.

lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::setMediaSourceEnded(bool)'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::reenqueueMediaIfNeeded(WTF::MediaTime const&, unsigned long, unsigned long)'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::addTrackBuffer(WTF::AtomString const&, WTF::RefPtr<WebCore::MediaDescription, WTF::RawPtrTraits<WebCore::MediaDescription>, WTF::DefaultRefDerefTraits<WebCore::MediaDescription> >&&)'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::resetTrackBuffers()'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::clearTrackBuffers()'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::setAllTrackBuffersNeedRandomAccess()'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::removeCodedFrames(WTF::MediaTime const&, WTF::MediaTime const&, WTF::MediaTime const&, bool, WTF::CompletionHandler<void ()>&&)'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::resetTimestampOffsetInTrackBuffers()'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::seekToTime(WTF::MediaTime const&)'
lib/libWebCoreTestSupport.a(lib/../Source/WebCore/CMakeFiles/WebCoreTestSupport.dir/platform/mock/mediasource/MockSourceBufferPrivate.cpp.o):MockSourceBufferPrivate.cpp:vtable for WebCore::MockSourceBufferPrivate: error: undefined reference to 'WebCore::SourceBufferPrivate::updateTrackIds(WTF::Vector<std::pair<WTF::AtomString, WTF::AtomString>, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&&)'
Comment 4 EWS 2021-02-09 19:25:05 PST
Committed r272632: <https://commits.webkit.org/r272632>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 419757 [details].
Comment 5 Radar WebKit Bug Importer 2021-02-10 14:28:17 PST
<rdar://problem/74206399>