Bug 208778

Summary:

Bounce displayWasRefreshed() via EventDispatcher

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

New Bugs

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

koivisto, simon.fraser, thorton, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

216296

Bug Blocks:

Attachments:

Description	Flags
Patch	koivisto: review+
Patch	none

Description Simon Fraser (smfr) 2020-03-07 16:38:18 PST

Bounce displayWasRefreshed() via EventDispatcher

Comment 1 Simon Fraser (smfr) 2020-03-07 16:40:20 PST

Created attachment 392912 [details]
Patch

Comment 2 Antti Koivisto 2020-03-07 22:34:47 PST

Comment on attachment 392912 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=392912&action=review

> Source/WebKit/WebProcess/WebProcess.h:342
> +#if ENABLE(WEBPROCESS_WINDOWSERVER_BLOCKING)
> +    void displayWasRefreshed(WebCore::PlatformDisplayID);
> +#endif

Please remove this and just call DisplayRefreshMonitorManager directly. Also remove the associated WebProcess message.

Comment 3 Radar WebKit Bug Importer 2020-04-23 15:40:47 PDT

<rdar://problem/62266522>

Comment 4 Simon Fraser (smfr) 2020-04-23 15:46:18 PDT

Created attachment 397394 [details]
Patch

Comment 5 EWS 2020-04-23 18:08:41 PDT

Committed r260614: <https://trac.webkit.org/changeset/260614>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 397394 [details].