Bug 228761

Summary: Use Observer in place of VideoFullscreenManagerProxyClient
Product: WebKit Reporter: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Component: MediaAssignee: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Status: RESOLVED FIXED    
Severity: Normal CC: jer.noble, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=228740
Attachments:
Description Flags
Patch none

Jean-Yves Avenard [:jya]
Reported 2021-08-03 17:22:31 PDT
This will be a nicer fix to bug 228740. We could use a WTF::Observer to notify clients that the PiP/FullScreen state has changed without having to worry about set/clearing clients.
Attachments
Patch (13.37 KB, patch)
2021-08-03 21:46 PDT, Jean-Yves Avenard [:jya] 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-08-03 17:24:01 PDT
<rdar://problem/81489026>
Jean-Yves Avenard [:jya]
Comment 2 2021-08-03 21:46:26 PDT
Created attachment 434884 [details] Patch
EWS
Comment 3 2021-08-04 16:47:53 PDT
Committed r280670 (240275@main): <https://commits.webkit.org/240275@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434884 [details].
Note You need to log in before you can comment on or make changes to this bug.