Bug 256669 - Always send displayDidRefresh via EventDispatcher
Summary: Always send displayDidRefresh via EventDispatcher
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matt Woodrow
URL:
Keywords: InRadar
Depends on:
Blocks: 232918
  Show dependency tree / graph
 
Reported: 2023-05-11 15:25 PDT by Matt Woodrow
Modified: 2023-05-30 14:17 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Woodrow 2023-05-11 15:25:39 PDT 
displayDidRefresh is currently sent over EventDispatcher, or WebProcess, depending on which consumers are registered.

Bug 232918 is going to add another consumer that wants the background-thread EventDispatcher behaviour, so it's simpler to just always do that.
Comment 1 Matt Woodrow 2023-05-11 15:35:06 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/13777
Comment 2 Radar WebKit Bug Importer 2023-05-18 15:26:19 PDT 
<rdar://problem/109535049>
Comment 3 Matt Woodrow 2023-05-30 14:17:40 PDT 
This can be a performance regression when we're only using it on the main thread, closing.