284823 2024-12-17 09:16:57 -0800 [Win] requestAnimationFrame can't reach display refresh rate 2025-04-07 23:26:06 -0700 1 1 1 Unclassified WebKit Web Template Framework WebKit Nightly Build PC Windows 11 REOPENED https://bugs.webkit.org/show_bug.cgi?id=291217 InRadar P2 Normal --- 290655 291226 1 ian.grunert fujii commit-queue fujii randy.luecke webkit-bug-importer oldest_to_newest 2082362 0 ian.grunert 2024-12-17 09:16:57 -0800 Test site: https://www.mobzystems.com/online/measure-browser-animation-speed/ WebKit MiniBrowser gets about 35fps on this test, well below the 60fps on Firefox. SetTimer / WM_TIMER used within RunLoopWin has a granularity of about 10ms and is likely bottlenecking our ability to hit 60fps on requestAnimationFrame. Chromium uses MsgWaitForMultipleObjectsEx in it's run loop to deal with this limitation: https://source.chromium.org/chromium/chromium/src/+/main:base/message_loop/message_pump_win.cc;l=467-485?q=WM_TIMER&ss=chromium%2Fchromium%2Fsrc Adopting something similar in WebKit's RunLoopWin would improve performance for WebGL apps such as https://webglsamples.org/aquarium/aquarium.html Seems like bug 211711 may be related. 2083801 1 webkit-bug-importer 2024-12-24 09:17:13 -0800 <rdar://problem/141996589> 2105604 2 ian.grunert 2025-03-24 09:40:46 -0700 Pull request: https://github.com/WebKit/WebKit/pull/42936 2106920 3 ews-feeder 2025-03-28 12:51:42 -0700 Committed 292846@main (70d5b92d1370): <https://commits.webkit.org/292846@main> Reviewed commits have been landed. Closing PR #42936 and removing active labels. 2106958 4 commit-queue 2025-03-28 15:18:23 -0700 Re-opened since this is blocked by bug 290655 2107101 5 fujii 2025-03-29 05:53:13 -0700 Pull request: https://github.com/WebKit/WebKit/pull/43244 2107638 6 ews-feeder 2025-03-31 16:59:59 -0700 Committed 292977@main (b376a535708e): <https://commits.webkit.org/292977@main> Reviewed commits have been landed. Closing PR #43244 and removing active labels. 2109533 7 commit-queue 2025-04-07 14:38:28 -0700 Re-opened since this is blocked by bug 291226 2109649 8 fujii 2025-04-07 23:26:06 -0700 I think RunLoop should have two modes for Windows. One mode uses WM_TIMER for UI process main thread. Another mode uses MsgWaitForMultipleObjectsEx for other thread.