RESOLVED FIXED314130
[Site Isolation] Introduce frameProcessForBackForwardNavigation helper
https://bugs.webkit.org/show_bug.cgi?id=314130
Summary [Site Isolation] Introduce frameProcessForBackForwardNavigation helper
Basuke Suzuki
Reported 2026-05-05 15:34:32 PDT
WebPageProxy::processForTheFrameItem() and WebPageProxy::copyFrameStateForBackForwardNavigation() duplicate the same Site Isolation gate logic to decide whether to use the live frame's process or fall back to the legacy main frame process. Consolidate this into a shared helper WebPageProxy::frameProcessForBackForwardNavigation(frameItem). The helper: - Returns nullptr when Site Isolation is disabled (the two callers fall back to their default). - Returns nullptr when the target item has a SuspendedPageProxy (that path is handled by the BFCache restoration logic — avoid picking a stale process here). - Returns nullptr with a RELEASE_LOG_ERROR if the frame's process is no longer reachable via a RemotePageProxy in the BrowsingContextGroup — a defensive safety-net. - Otherwise returns the live frame's process. Both existing call sites are simplified to use the helper. Prep patch for rdar://173743733 (multi-process BFCache restoration under Site Isolation).
Attachments
Basuke Suzuki
Comment 1 2026-05-05 17:05:23 PDT
Basuke Suzuki
Comment 2 2026-05-05 17:05:42 PDT
EWS
Comment 3 2026-05-06 11:47:03 PDT
Committed 312724@main (e3e8e6850c8d): <https://commits.webkit.org/312724@main> Reviewed commits have been landed. Closing PR #64314 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.