[blink][xr] Add a XRLayerHolder struct to combine LayerId and image ref [chromium/src : main]

0 views

Skip to first unread message

Mingjing Zhang (Gerrit)

unread,

Mar 4, 2026, 11:34:57 AM (12 days ago) Mar 4

to Vasiliy Telezhnikov, AI Code Reviewer, Chromium LUCI CQ, chromium...@chromium.org, blink-...@chromium.org, feature-v...@chromium.org

Attention needed from Vasiliy Telezhnikov

Mingjing Zhang added 1 comment

Patchset-level comments

File-level comment, Patchset 4 (Latest):

Mingjing Zhang . resolved

PTAL. Thanks.

Open in Gerrit

Related details

Attention is currently required from:

Vasiliy Telezhnikov

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

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

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Vasiliy Telezhnikov (Gerrit)

unread,

Mar 4, 2026, 2:31:15 PM (11 days ago) Mar 4

to Mingjing Zhang, AI Code Reviewer, Chromium LUCI CQ, chromium...@chromium.org, blink-...@chromium.org, feature-v...@chromium.org

Attention needed from Mingjing Zhang

Vasiliy Telezhnikov voted and added 2 comments

Votes added by Vasiliy Telezhnikov

Code-Review

2 comments

Patchset-level comments

File-level comment, Patchset 4 (Latest):

Vasiliy Telezhnikov . resolved

lgtm % nit, thanks.

File third_party/blink/renderer/modules/xr/xr_frame_provider.h

Line 38, Patchset 4 (Latest):struct XRLayerHolder {

Vasiliy Telezhnikov . unresolved

nit: XRLayerUpdate, as essentially we're tracking update to each layer.