[webkit-changes] [WebKit/WebKit] b44284: Have MediaDescription stop using AtomString
Jean-Yves Avenard
noreply at github.com
Thu Feb 8 01:49:03 PST 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: b4428489b41ebbeaa275341b74aec164fb693faa
https://github.com/WebKit/WebKit/commit/b4428489b41ebbeaa275341b74aec164fb693faa
Author: Jean-Yves Avenard <jya at apple.com>
Date: 2024-02-08 (Thu, 08 Feb 2024)
Changed paths:
M Source/WebCore/Modules/mediasource/SourceBuffer.cpp
M Source/WebCore/platform/MediaDescription.h
M Source/WebCore/platform/graphics/avfoundation/objc/SourceBufferParserAVFObjC.mm
M Source/WebCore/platform/graphics/cocoa/SourceBufferParserWebM.cpp
M Source/WebCore/platform/graphics/gstreamer/mse/GStreamerMediaDescription.cpp
M Source/WebCore/platform/graphics/gstreamer/mse/GStreamerMediaDescription.h
M Source/WebCore/platform/mock/mediasource/MockBox.h
M Source/WebCore/platform/mock/mediasource/MockSourceBufferPrivate.cpp
M Source/WebKit/GPUProcess/media/MediaDescriptionInfo.h
M Source/WebKit/GPUProcess/media/MediaDescriptionInfo.serialization.in
M Source/WebKit/GPUProcess/media/RemoteMediaDescription.h
Log Message:
-----------
Have MediaDescription stop using AtomString
https://bugs.webkit.org/show_bug.cgi?id=268728
rdar://122292724
Reviewed by Youenn Fablet.
Sub-task of 265982.
To be used in WorkQueues, MediaDescription needs to stop using AtomString.
Instead we make this object use ThreadSafeRefCount, and return a StringView
which explicitly enforce that you can't extend the lifetime of the string
outside the lifetime of the MediaDescription container.
No change in observable behaviour.
* Source/WebCore/Modules/mediasource/SourceBuffer.cpp:
(WebCore::SourceBuffer::sourceBufferPrivateDidReceiveInitializationSegment):
(WebCore::SourceBuffer::validateInitializationSegment):
* Source/WebCore/platform/MediaDescription.h:
* Source/WebCore/platform/graphics/avfoundation/objc/SourceBufferParserAVFObjC.mm:
* Source/WebCore/platform/graphics/cocoa/SourceBufferParserWebM.cpp:
* Source/WebCore/platform/graphics/gstreamer/mse/GStreamerMediaDescription.cpp:
(WebCore::GStreamerMediaDescription::codec const):
(WebCore::GStreamerMediaDescription::extractCodecName const):
(WebCore::GStreamerMediaDescription::extractCodecName): Deleted.
* Source/WebCore/platform/graphics/gstreamer/mse/GStreamerMediaDescription.h:
(WebCore::GStreamerMediaDescription::GStreamerMediaDescription):
* Source/WebCore/platform/mock/mediasource/MockBox.h:
* Source/WebCore/platform/mock/mediasource/MockSourceBufferPrivate.cpp:
* Source/WebKit/GPUProcess/media/MediaDescriptionInfo.h:
(WebKit::MediaDescriptionInfo::MediaDescriptionInfo):
* Source/WebKit/GPUProcess/media/MediaDescriptionInfo.serialization.in:
* Source/WebKit/GPUProcess/media/RemoteMediaDescription.h:
(WebKit::RemoteMediaDescription::RemoteMediaDescription):
Canonical link: https://commits.webkit.org/274276@main
More information about the webkit-changes
mailing list