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   

Description Jean-Yves Avenard [:jya] 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.
Comment 1 Radar WebKit Bug Importer 2021-06-23 22:22:07 PDT
<rdar://problem/79709759>