Bug 227338

Summary: Two VideoToolboxSoftLink.h
Product: WebKit Reporter: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Component: MediaAssignee: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Status: NEW    
Severity: Normal CC: webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Jean-Yves Avenard [:jya]
Reported 2021-06-23 22:21:47 PDT
There's currently two VideoToolboxSoftLink.h One in WebCore/PAL/pal/cf/VideoToolboxSoftLink.h the other in WebCore/platform/cocoa/VideoToolboxSoftLink.h Both soft-link the VideoToolbox framework, but with different methods. cf/VideoToolboxSoftLink.h mostly targets video encoder while cocoa/VideoToolboxSoftLink.h targes video decoding session. Method in the first are added in the PAL namespace while the latter use WebCore. Sometimes both are used at the same time. This is confusing, we should merge them, and for consistency have all soft links made under PAL namespace.
Attachments
Radar WebKit Bug Importer
Comment 1 2021-06-23 22:22:07 PDT
Note You need to log in before you can comment on or make changes to this bug.