Bug 221625 - [GTK] Export additional methods of SourceBufferPrivate to support hidden visibility
Summary: [GTK] Export additional methods of SourceBufferPrivate to support hidden visi...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Don Olmstead
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-02-09 13:41 PST by Don Olmstead
Modified: 2021-02-10 14:28 PST (History)
2 users (show)

See Also:


Attachments
Patch (4.05 KB, patch)
2021-02-09 13:48 PST, Don Olmstead
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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>