Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: cb220370b4bebedaa47a9b40eee035e95cc4ca2d
https://github.com/WebKit/WebKit/commit/cb220370b4bebedaa47a9b40eee035e95cc4ca2d
Author: Charlie Wolfe <[email protected]>
Date: 2026-06-17 (Wed, 17 Jun 2026)
Changed paths:
M Source/WebKit/WebProcess/GPU/media/RemoteAudioDestinationProxy.cpp
Log Message:
-----------
RemoteAudioDestinationProxy::startRenderingThread is off by one when the
frame counter wraps
https://bugs.webkit.org/show_bug.cgi?id=317222
Reviewed by Chris Dumez.
The wraparound branch was unnecessary. Unsigned subtraction is defined modulo
2^32, so
totalFrameCount - m_lastFrameCount already yields the correct elapsed frame
count across a 32-bit
counter wrap. Drop the conditional entirely. (The original branch was also off
by one: it used
numeric_limits<uint32_t>::max() - m_lastFrameCount, which is one short of the
2^32 - m_lastFrameCount
needed.)
* Source/WebKit/WebProcess/GPU/media/RemoteAudioDestinationProxy.cpp:
(WebKit::RemoteAudioDestinationProxy::startRenderingThread):
Canonical link: https://commits.webkit.org/315395@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications