[WebXR] Check page visibility during session request validation [chromium/src : main]

0 views
Skip to first unread message

Alexander Cooper (Gerrit)

unread,
May 29, 2026, 3:58:17 PM (4 days ago) May 29
to Brandon Jones, Chromium LUCI CQ, chromium...@chromium.org, feature-v...@chromium.org
Attention needed from Brandon Jones

Alexander Cooper voted and added 1 comment

Votes added by Alexander Cooper

Auto-Submit+1

1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Alexander Cooper . resolved

Brandon PTAL

Open in Gerrit

Related details

Attention is currently required from:
  • Brandon Jones
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
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: Ibcaafa8dbdbec3a53e556a9459d3658edc3ffa24
Gerrit-Change-Number: 7885957
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Cooper <alco...@chromium.org>
Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>
Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>
Gerrit-Attention: Brandon Jones <baj...@chromium.org>
Gerrit-Comment-Date: Fri, 29 May 2026 19:57:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Brandon Jones (Gerrit)

unread,
Jun 1, 2026, 6:40:36 PM (yesterday) Jun 1
to Alexander Cooper, Chromium LUCI CQ, chromium...@chromium.org, feature-v...@chromium.org
Attention needed from Alexander Cooper

Brandon Jones voted

Code-Review+1
Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Alexander Cooper
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    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: Ibcaafa8dbdbec3a53e556a9459d3658edc3ffa24
    Gerrit-Change-Number: 7885957
    Gerrit-PatchSet: 1
    Gerrit-Owner: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>
    Gerrit-Attention: Alexander Cooper <alco...@chromium.org>
    Gerrit-Comment-Date: Mon, 01 Jun 2026 22:40:23 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Alexander Cooper (Gerrit)

    unread,
    Jun 1, 2026, 7:00:42 PM (yesterday) Jun 1
    to Brandon Jones, Chromium LUCI CQ, chromium...@chromium.org, feature-v...@chromium.org
    Attention needed from Brandon Jones

    Alexander Cooper voted Commit-Queue+2

    Commit-Queue+2
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Brandon Jones
    Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    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: Ibcaafa8dbdbec3a53e556a9459d3658edc3ffa24
    Gerrit-Change-Number: 7885957
    Gerrit-PatchSet: 1
    Gerrit-Owner: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>
    Gerrit-Attention: Brandon Jones <baj...@chromium.org>
    Gerrit-Comment-Date: Mon, 01 Jun 2026 23:00:27 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Chromium LUCI CQ (Gerrit)

    unread,
    Jun 1, 2026, 8:17:10 PM (yesterday) Jun 1
    to Alexander Cooper, Brandon Jones, chromium...@chromium.org, feature-v...@chromium.org

    Chromium LUCI CQ submitted the change

    Change information

    Commit message:
    [WebXR] Check page visibility during session request validation

    Validates that the requesting frame's visibility state is set to
    visible when processing a VRService::RequestSession Mojo call in
    VRServiceImpl, maintaining proper alignment with renderer-side
    session requirements.
    Fixed: 517598518
    Change-Id: Ibcaafa8dbdbec3a53e556a9459d3658edc3ffa24
    Auto-Submit: Alexander Cooper <alco...@chromium.org>
    Commit-Queue: Alexander Cooper <alco...@chromium.org>
    Reviewed-by: Brandon Jones <baj...@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1639800}
    Files:
    • M content/browser/xr/service/vr_service_impl.cc
    Change size: S
    Delta: 1 file changed, 12 insertions(+), 0 deletions(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Brandon Jones
    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: Ibcaafa8dbdbec3a53e556a9459d3658edc3ffa24
    Gerrit-Change-Number: 7885957
    Gerrit-PatchSet: 2
    Gerrit-Owner: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>
    Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>
    open
    diffy
    satisfied_requirement
    Reply all
    Reply to author
    Forward
    0 new messages