Skip to content

Commit e2da729

Browse files
committed
parseSequenceHeaderOBU should return a AV1CodecConfigurationRecord
https://bugs.webkit.org/show_bug.cgi?id=304108 rdar://166439682 Reviewed by Youenn Fablet. Replace ParsedSequenceHeaderParameters with AV1CodecConfigurationRecord Amend AV1CodecConfigurationRecord to store video dimensions Have parseAV1DecoderConfigurationRecord take a span and use a BitReader instead of a JSC::DataView This avoids two memory allocations. Amend parseSequenceHeaderOBU to fully decode the Sequence Header OBU and retrieved the full colour profile when possible. We'll use this method in a future change to create a VideoInfo structure where full colour profile is required. * Source/WebCore/platform/graphics/AV1Utilities.cpp: (WebCore::parseAV1DecoderConfigurationRecord): (WebCore::parseSequenceHeaderOBU): * Source/WebCore/platform/graphics/AV1Utilities.h: * Source/WebCore/platform/graphics/cocoa/AV1UtilitiesCocoa.h: * Source/WebCore/platform/graphics/cocoa/AV1UtilitiesCocoa.mm: (WebCore::readULEBSize): (WebCore::getSequenceHeaderOBU): (WebCore::computeAV1InputFormat): * Source/WebCore/platform/video-codecs/cocoa/RTCVideoDecoderVTBAV1.mm: (BitReader::BitReader): Deleted. (BitReader::read): Deleted. (BitReader::readBit): Deleted. (readULEBSize): Deleted. (getSequenceHeaderOBU): Deleted. (parseSequenceHeaderOBU): Deleted. (computeAV1InputFormat): Deleted. Canonical link: https://commits.webkit.org/304474@main
1 parent a6503a7 commit e2da729

File tree

6 files changed

+684
-282
lines changed

6 files changed

+684
-282
lines changed

0 commit comments

Comments
 (0)