Bug 237200

Summary: Ensure RemoteMediaPlayerProxy sets its resource owner to pixel buffers created by its player
Product: WebKit Reporter: youenn fablet <youennf>
Component: MediaAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing ews-feeder: commit-queue-

Description youenn fablet 2022-02-25 04:58:26 PST
Ensure RemoteMediaPlayerProxy sets its resource owner to pixel buffers created by its player
Comment 1 youenn fablet 2022-02-25 05:02:38 PST
Created attachment 453200 [details]
Patch
Comment 2 Eric Carlson 2022-02-25 08:50:15 PST
Comment on attachment 453200 [details]
Patch

It might be worth adding the same logic to MediaPlayerPrivateMediaStreamAVFObjC as well, in case we move it to the GPU Process at some point.
Comment 3 youenn fablet 2022-02-27 23:15:50 PST
(In reply to Eric Carlson from comment #2)
> Comment on attachment 453200 [details]
> Patch
> 
> It might be worth adding the same logic to
> MediaPlayerPrivateMediaStreamAVFObjC as well, in case we move it to the GPU
> Process at some point.

I'll add an ASSERT in MediaPlayerPrivateMediaStreamAVFObjC to make it clear it should be implemented if we ever do that.
Comment 4 youenn fablet 2022-02-27 23:21:38 PST
Created attachment 453370 [details]
Patch for landing
Comment 5 EWS 2022-02-28 02:05:08 PST
Committed r290590 (247868@main): <https://commits.webkit.org/247868@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453370 [details].
Comment 6 Radar WebKit Bug Importer 2022-02-28 02:06:18 PST
<rdar://problem/89548310>