Bug 228761

Summary:

Use Observer in place of VideoFullscreenManagerProxyClient

Product:

WebKit

Reporter:

Jean-Yves Avenard [:jya] <jean-yves.avenard>

Component:

Media

Assignee:

Jean-Yves Avenard [:jya] <jean-yves.avenard>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

jer.noble, webkit-bug-importer

Priority:

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

Description Jean-Yves Avenard [:jya] 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.

Comment 1 Radar WebKit Bug Importer 2021-08-03 17:24:01 PDT

<rdar://problem/81489026>

Comment 2 Jean-Yves Avenard [:jya] 2021-08-03 21:46:26 PDT

Created attachment 434884 [details]
Patch

Comment 3 EWS 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].