Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 5f233ab494317a4a25569e364dc5f5d1b284b667
https://github.com/WebKit/WebKit/commit/5f233ab494317a4a25569e364dc5f5d1b284b667
Author: Yijia Huang <[email protected]>
Date: 2026-03-13 (Fri, 13 Mar 2026)
Changed paths:
M Source/JavaScriptCore/jsc.cpp
M Source/JavaScriptCore/wasm/debugger/testwasmdebugger.cpp
M Source/WTF/wtf/PlatformEnable.h
M Source/WebKit/WebProcess/WebProcess.cpp
Log Message:
-----------
[JSC][WASM][Debugger] ENABLE(WEBASSEMBLY_DEBUGGER) must not be
architecture-dependent to keep IPC MessageNames consistent
https://bugs.webkit.org/show_bug.cgi?id=309905
rdar://172097502
Reviewed by Yusuke Suzuki and Mark Lam.
Safari crashes under Rosetta because ENABLE(WEBASSEMBLY_DEBUGGER) was gated on
CPU(ARM64),
causing the SendWasmDebuggerResponse IPC message to be absent from the
MessageNames enum on
x86_64 builds. ARM64 WebContent processes and the Intel Safari UI process then
disagree on
message indices, causing Safari to kill the sender upon receiving an "invalid"
message.
Fix by removing CPU(ARM64) from ENABLE_WEBASSEMBLY_DEBUGGER in PlatformEnable.h
so the IPC
plumbing compiles on all Mac architectures. The debugger engine itself is still
restricted to
ARM64 by guarding the DebugServer::start() and startRWI() call sites with
CPU(ARM64), and
testwasmdebugger skips all tests on non-ARM64.
Canonical link: https://commits.webkit.org/309253@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
