Skip to content

Upgrade to upstream WebKit 6b58d86abe - #651

Closed
robobun wants to merge 226 commits into
mainfrom
bun/upgrade-to-6b58d86abe
Closed

robobun wants to merge 226 commits into
mainfrom
bun/upgrade-to-6b58d86abe

Merge remote-tracking branch 'origin/main' into bun/upgrade-to-6b58d8…

2ed8525
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Sep 23, 2026 in 56m 11s

Code review found 4 important issues

Found 9 candidates, confirmed 6. See review comments for details.

Details

Severity Count
🔴 Important 4
🟡 Nit 1
🟣 Pre-existing 1
Severity File:Line Issue
🔴 Important Source/WebKit/NetworkProcess/NetworkLoad.cpp:206 The network process can use freed memory when a throttled response is cancelled from inside a data callback, which the b
🔴 Important Source/WebKit/NetworkProcess/NetworkLoad.cpp:559 Changing emulated latency while loads are in flight now resumes tasks that are already running or still queued in the sc
🔴 Important Source/WebInspectorUI/UserInterface/Controllers/NetworkManager.js:1574 Web Inspector users inspecting an iOS 16.4-17.4 or macOS 13.3-14.4 backend can no longer apply or clear network throttli
🔴 Important Source/WebKit/NetworkProcess/NetworkLoad.cpp:110 With bandwidth throttling on, a multipart/x-mixed-replace stream (e.g. an MJPEG camera page) gets the tail of one part a
🟡 Nit Source/WebCore/css/parser/CSSSelectorParser.cpp:799 nit: Authors animating an SVG element's class with SMIL now see '[class~="x"]' rules toggle with the animated class list
🟣 Pre-existing Source/JavaScriptCore/runtime/JSBigInt.cpp:3185 Pre-existing: terminating a VM (worker.terminate) during a large BigInt '%' crashes the process on a RELEASE_ASSERT inst