Bug 256399

Summary:	[UI-side compositing] Guard against UI process hangs caused by incorrect frame rate data
Product:	WebKit	Reporter:	Simon Fraser (smfr) <simon.fraser>
Component:	Scrolling	Assignee:	Simon Fraser (smfr) <simon.fraser>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	simon.fraser, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Local Build
Hardware:	Unspecified
OS:	Unspecified

Simon Fraser (smfr)

Reported 2023-05-05 15:02:56 PDT

There's some evidence of a UI-process (i.e. Safari) hang caused by RemoteLayerTreeEventDispatcher::waitForRenderingUpdateCompletionOrTimeout() taking too long. This could happen if m_stateCondition.waitUntil() has a long timeout, so impose a cap of 32ms on that timeout.

Attachments
Add attachment proposed patch, testcase, etc.

Simon Fraser (smfr)

Comment 1 2023-05-05 15:03:16 PDT

<rdar://108954321>

Simon Fraser (smfr)

Comment 2 2023-05-05 15:09:42 PDT

Pull request: https://github.com/WebKit/WebKit/pull/13525

EWS

Comment 3 2023-05-08 11:18:39 PDT

Committed 263806@main (3f7af1859b92): <https://commits.webkit.org/263806@main> Reviewed commits have been landed. Closing PR #13525 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.