[Partitioned Popins] (6) Add permissions policy for popin [chromium/src : main]

0 views
Skip to first unread message

Ari Chivukula (Gerrit)

unread,
Aug 21, 2024, 9:47:53 AM8/21/24
to Daniel Cheng, AyeAye, Chromium LUCI CQ, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Daniel Cheng

Ari Chivukula voted Auto-Submit+1

Auto-Submit+1
Open in Gerrit

Related details

Attention is currently required from:
  • Daniel Cheng
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 3
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Luna Lu <loon...@chromium.org>
Gerrit-Attention: Daniel Cheng <dch...@chromium.org>
Gerrit-Comment-Date: Wed, 21 Aug 2024 13:47:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Daniel Cheng (Gerrit)

unread,
Aug 21, 2024, 12:29:54 PM8/21/24
to Ari Chivukula, Daniel Cheng, AyeAye, Chromium LUCI CQ, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revi...@chromium.org, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Ari Chivukula

Daniel Cheng voted and added 1 comment

Votes added by Daniel Cheng

Code-Review+1

1 comment

Patchset-level comments
File-level comment, Patchset 7 (Latest):
Daniel Cheng . resolved

LGTM

Open in Gerrit

Related details

Attention is currently required from:
  • Ari Chivukula
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 7
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Luna Lu <loon...@chromium.org>
Gerrit-Attention: Ari Chivukula <ari...@chromium.org>
Gerrit-Comment-Date: Wed, 21 Aug 2024 16:29:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Ari Chivukula (Gerrit)

unread,
Aug 21, 2024, 12:30:51 PM8/21/24
to Daniel Cheng, AyeAye, Chromium LUCI CQ, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revi...@chromium.org, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Ari Chivukula voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 7
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Luna Lu <loon...@chromium.org>
Gerrit-Comment-Date: Wed, 21 Aug 2024 16:30:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Blink W3C Test Autoroller (Gerrit)

unread,
Aug 21, 2024, 12:44:57 PM8/21/24
to Ari Chivukula, Daniel Cheng, AyeAye, Chromium LUCI CQ, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revi...@chromium.org, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Message from Blink W3C Test Autoroller

Exportable changes to web-platform-tests were detected in this CL and a pull request in the upstream repo has been made: https://github.com/web-platform-tests/wpt/pull/47719.

When this CL lands, the bot will automatically merge the PR on GitHub if the required GitHub checks pass; otherwise, ecosystem-infra@ team will triage the failures and may contact you.

WPT Export docs:
https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md#Automatic-export-process

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 7
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Luna Lu <loon...@chromium.org>
Gerrit-Comment-Date: Wed, 21 Aug 2024 16:44:45 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Aug 21, 2024, 12:45:51 PM8/21/24
to Ari Chivukula, Blink W3C Test Autoroller, Daniel Cheng, AyeAye, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revi...@chromium.org, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[Partitioned Popins] (6) Add permissions policy for popin

This CL adds a permissions policy to control access to the partitioned
popin feature and sets the default to 'self'.

This series of CLs implement core components of the Partitioned Popin
system, significant additional effort will be needed to align with the
explainer and I2P, but all of that will depend on this work:
(1) Implement `popin` window feature
(2) `popin` feature triggers tab modal popup
(3) `popin` feature triggers third-party storage partitioning
(4) Renderer awareness of popin top-origin
(5) Limit window.opener access for popin
(6) Add permissions policy for popin

Explainer: https://explainers-by-googlers.github.io/partitioned-popins/
I2P: https://groups.google.com/a/chromium.org/g/blink-dev/c/ApU_zUmpQ2g/
Bug: 340606651
Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Auto-Submit: Ari Chivukula <ari...@chromium.org>
Reviewed-by: Daniel Cheng <dch...@chromium.org>
Commit-Queue: Ari Chivukula <ari...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1344862}
Files:
  • M third_party/blink/public/devtools_protocol/browser_protocol.pdl
  • M third_party/blink/public/mojom/permissions_policy/permissions_policy_feature.mojom
  • M third_party/blink/renderer/core/frame/local_dom_window.cc
  • M third_party/blink/renderer/core/permissions_policy/permissions_policy_features.json5
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/partitioned-popins.permission-all.tentative.sub.https.window.js
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/partitioned-popins.permission-all.tentative.sub.https.window.js.headers
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/partitioned-popins.permission-default.tentative.sub.https.window.js
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/partitioned-popins.permission-self.tentative.sub.https.window.js
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/partitioned-popins.permission-self.tentative.sub.https.window.js.headers
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/resources/partitioned-popins.permissions-iframe.html
  • A third_party/blink/web_tests/external/wpt/partitioned-popins/resources/partitioned-popins.permissions-popin.html
  • M third_party/blink/web_tests/webexposed/feature-policy-features-expected.txt
  • M tools/metrics/histograms/enums.xml
Change size: M
Delta: 13 files changed, 152 insertions(+), 0 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Daniel Cheng
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 8
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
open
diffy
satisfied_requirement

Blink W3C Test Autoroller (Gerrit)

unread,
Aug 21, 2024, 1:15:31 PM8/21/24
to Ari Chivukula, Chromium LUCI CQ, Daniel Cheng, AyeAye, Chromium Metrics Reviews, chromium...@chromium.org, devtools...@chromium.org, Luna Lu, blink-revi...@chromium.org, blink-revie...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, feature-co...@chromium.org, iclella...@chromium.org, ipc-securi...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Message from Blink W3C Test Autoroller

The WPT PR for this CL has been merged upstream! https://github.com/web-platform-tests/wpt/pull/47719

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ifde9f8e3529d794b5242df561603df7739a1f892
Gerrit-Change-Number: 5802591
Gerrit-PatchSet: 8
Gerrit-Owner: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Ari Chivukula <ari...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Daniel Cheng <dch...@chromium.org>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Luna Lu <loon...@chromium.org>
Gerrit-Comment-Date: Wed, 21 Aug 2024 17:15:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages