[M135] Relax the COI requirement for Direct Sockets in workers [chromium/src : refs/branch-heads/7049]

0 views

Skip to first unread message

Simon Hangl (Gerrit)

unread,

Apr 8, 2025, 2:52:32 PM4/8/25

to Andrew Rayskiy, AyeAye, rmcelra...@chromium.org, blink-...@chromium.org

Attention needed from Andrew Rayskiy

Simon Hangl voted and added 1 comment

Votes added by Simon Hangl

Code-Review

1 comment

Patchset-level comments

File-level comment, Patchset 2 (Latest):

Simon Hangl . resolved

LGTM

Open in Gerrit

Related details

Attention is currently required from:

Andrew Rayskiy

Submit Requirements:

Code-Review
Lint

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Andrew Rayskiy (Gerrit)

unread,

Apr 9, 2025, 4:52:14 AM4/9/25

to Simon Hangl, AyeAye, rmcelra...@chromium.org, blink-...@chromium.org

Andrew Rayskiy voted Commit-Queue+2

Commit-Queue

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Review
Lint

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Chromium LUCI CQ (Gerrit)

unread,

Apr 9, 2025, 5:43:18 AM4/9/25

to Andrew Rayskiy, Simon Hangl, AyeAye, rmcelra...@chromium.org, blink-...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:

[M135] Relax the COI requirement for Direct Sockets in workers

Chrome Apps are allowed to use Direct Sockets even without specifying
cross-origin isolation in the manifest; for this reason we have to
check just the permissions-policy feature and not the entire capability.

(cherry picked from commit b4d6430bfe1dc166187fa975d3e8146f5bbc3cef)

Bug: 409051903

Change-Id: Ie1d90879d5fc954c3f749b1dcba1d11dd4bdd1f0

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6434162

Commit-Queue: Andrew Rayskiy <green...@google.com>

Auto-Submit: Andrew Rayskiy <green...@google.com>

Reviewed-by: Simon Hangl <sim...@google.com>

Cr-Original-Commit-Position: refs/heads/main@{#1443405}

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6439422

Cr-Commit-Position: refs/branch-heads/7049@{#1796}

Cr-Branched-From: 2dab7846d0951a552bdc4f350dad497f986e6fed-refs/heads/main@{#1427262}

Files:

M chrome/browser/direct_sockets/direct_sockets_apitest.cc
M third_party/blink/renderer/modules/direct_sockets/socket.cc

Change size: M

Delta: 2 files changed, 80 insertions(+), 7 deletions(-)

Branch: refs/branch-heads/7049

Submit Requirements:

Code-Review: +1 by Simon Hangl

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages