fix: inherit sandbox flags in windows opened by a sandboxed top-level frame by trop[bot] · Pull Request #52848 · electron/electron · GitHub
Skip to content

fix: inherit sandbox flags in windows opened by a sandboxed top-level frame - #52848

Merged
MarshallOfSound merged 1 commit into
43-x-yfrom
trop/43-x-y-bp-fix-inherit-sandbox-flags-in-windows-opened-by-a-sandboxed-top-level-frame-1786876176089
Aug 16, 2026
Merged

MarshallOfSound merged 1 commit into
43-x-yfrom
trop/43-x-y-bp-fix-inherit-sandbox-flags-in-windows-opened-by-a-sandboxed-top-level-frame-1786876176089

Conversation

@trop

@trop trop Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Backport of #52841

See that PR for details.

Notes: Fixed windows opened by a sandboxed top-level frame not inheriting the opener's sandbox restrictions.

… frame

WebContents::OpenURLFromTab only propagated the initiating frame's sandbox
flags to the window it created when that frame had a parent. A top-level
frame can carry a non-empty sandboxing flag set as well (via a sandboxed
iframe's popup, or a Content-Security-Policy: sandbox header), so resolve
and apply the initiator's active sandbox flags regardless of its position
in the frame tree.

Co-authored-by: Sam Attard <sattard@anthropic.com>
@trop trop Bot added 43-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Aug 16, 2026
@MarshallOfSound
MarshallOfSound enabled auto-merge (squash) August 16, 2026 10:31
@MarshallOfSound
MarshallOfSound merged commit 6594d5a into 43-x-y Aug 16, 2026
96 checks passed
@MarshallOfSound
MarshallOfSound deleted the trop/43-x-y-bp-fix-inherit-sandbox-flags-in-windows-opened-by-a-sandboxed-top-level-frame-1786876176089 branch August 16, 2026 12:53
@release-clerk

release-clerk Bot commented Aug 16, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

43-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant