Add XRMediaBinding IDL and stubs [chromium/src : main]

[Alexander Cooper (Gerrit)]

Attention needed from Brandon Jones and Yong Li (xWF)

Alexander Cooper added 1 comment

File third_party/blink/web_tests/virtual/stable/webexposed/global-interface-listing-expected.txt

Line 11957, Patchset 6 (Latest):interface XRMediaBinding
attribute @@toStringTag
method constructor
method createCylinderLayer
method createEquirectLayer
method createQuadLayer

Alexander Cooper . unresolved

We should put this behind a runtime_enabled_feature so that it doesn't appear in virtual/stable yet

Open in Gerrit

Related details

Attention is currently required from:

• Brandon Jones
• Yong Li (xWF)

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• No-Unresolved-Comments
• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 6

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Comment-Date: Thu, 21 May 2026 19:50:10 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

[Yong Li (xWF) (Gerrit)]

Attention needed from Alexander Cooper and Brandon Jones

Yong Li (xWF) added 1 comment

File third_party/blink/web_tests/virtual/stable/webexposed/global-interface-listing-expected.txt

Line 11957, Patchset 6:interface XRMediaBinding


attribute @@toStringTag
method constructor
method createCylinderLayer
method createEquirectLayer
method createQuadLayer

Alexander Cooper . resolved

We should put this behind a runtime_enabled_feature so that it doesn't appear in virtual/stable yet

Yong Li (xWF)

Done

Open in Gerrit

Related details

Attention is currently required from:

• Alexander Cooper
• Brandon Jones

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

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Attention: Alexander Cooper <alco...@chromium.org>

Gerrit-Comment-Date: Tue, 26 May 2026 20:26:38 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

Comment-In-Reply-To: Alexander Cooper <alco...@chromium.org>

[Alexander Cooper (Gerrit)]

Attention needed from Brandon Jones and Yong Li (xWF)

Alexander Cooper added 1 comment

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6928, Patchset 7 (Latest): public: true,

Alexander Cooper . unresolved

Status should probably be experimental; which would at least expose the regular global-interface-listing-expected.txt entry.

Open in Gerrit

Related details

Attention is currently required from:

• Brandon Jones
• Yong Li (xWF)

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review

• No-Unresolved-Comments

• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Comment-Date: Tue, 26 May 2026 20:33:01 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

[Yong Li (xWF) (Gerrit)]

Attention needed from Alexander Cooper and Brandon Jones

Yong Li (xWF) added 1 comment

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6928, Patchset 7 (Latest): public: true,

Alexander Cooper . unresolved

Status should probably be experimental; which would at least expose the regular global-interface-listing-expected.txt entry.

Yong Li (xWF)

this just defines the IDLs. It cannot be experimental yet. Do you want me to change it to experimental in the implementation CL? It always enables the feature currently.

Open in Gerrit

Related details

Attention is currently required from:

• Alexander Cooper
• Brandon Jones

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• No-Unresolved-Comments
• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Attention: Alexander Cooper <alco...@chromium.org>

Gerrit-Comment-Date: Tue, 26 May 2026 22:11:21 +0000

[Alexander Cooper (Gerrit)]

Attention needed from Brandon Jones and Yong Li (xWF)

Alexander Cooper added 1 comment

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6928, Patchset 7 (Latest): public: true,

Alexander Cooper . unresolved

Status should probably be experimental; which would at least expose the regular global-interface-listing-expected.txt entry.

Yong Li (xWF)

this just defines the IDLs. It cannot be experimental yet. Do you want me to change it to experimental in the implementation CL? It always enables the feature currently.

Alexander Cooper

experimental just means it's in development.

Open in Gerrit

Related details

Attention is currently required from:

• Brandon Jones
• Yong Li (xWF)

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• No-Unresolved-Comments
• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Comment-Date: Tue, 26 May 2026 22:23:08 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

Comment-In-Reply-To: Yong Li (xWF) <yyon...@google.com>

Comment-In-Reply-To: Alexander Cooper <alco...@chromium.org>

[Brandon Jones (Gerrit)]

Attention needed from Yong Li (xWF)

Brandon Jones added 1 comment

File third_party/blink/renderer/modules/xr/xr_media_binding.idl

Line 12, Patchset 7 (Latest): [RaisesException] XRQuadLayer createQuadLayer(HTMLVideoElement video, optional XRMediaQuadLayerInit init = {});

Brandon Jones . unresolved

I recall that this came up with a previous CL, but these dictionaries (`XRMediaQuadLayerInit`, `XRMediaCylinderLayerInit`, etc) shouldn't be optional, and it's a bug in the spec forced by our tooling that they are listed as optional there. Can we make these required? Or is there a separate blink-side bug that prevents that?

Open in Gerrit

Related details

Attention is currently required from:

• Yong Li (xWF)

Gerrit-Comment-Date: Wed, 27 May 2026 18:02:19 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

[Yong Li (xWF) (Gerrit)]

Attention needed from Brandon Jones

Yong Li (xWF) added 1 comment

File third_party/blink/renderer/modules/xr/xr_media_binding.idl

Line 12, Patchset 7 (Latest): [RaisesException] XRQuadLayer createQuadLayer(HTMLVideoElement video, optional XRMediaQuadLayerInit init = {});

Brandon Jones . unresolved

I recall that this came up with a previous CL, but these dictionaries (`XRMediaQuadLayerInit`, `XRMediaCylinderLayerInit`, etc) shouldn't be optional, and it's a bug in the spec forced by our tooling that they are listed as optional there. Can we make these required? Or is there a separate blink-side bug that prevents that?

Yong Li (xWF)

Do you mean this is a bug? https://www.w3.org/TR/webxrlayers-1/#xrmediabinding
Any quick hint about how this spec gets generated? I feel it may be reasonable to be optional, because their members are all optional. So a default constructor should be fine.

Open in Gerrit

Related details

Attention is currently required from:

• Brandon Jones

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• No-Unresolved-Comments
• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Comment-Date: Wed, 27 May 2026 18:36:23 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

Comment-In-Reply-To: Brandon Jones <baj...@chromium.org>

[Brandon Jones (Gerrit)]

Attention needed from Yong Li (xWF)

Brandon Jones added 1 comment

File third_party/blink/renderer/modules/xr/xr_media_binding.idl

Line 12, Patchset 7 (Latest): [RaisesException] XRQuadLayer createQuadLayer(HTMLVideoElement video, optional XRMediaQuadLayerInit init = {});

Brandon Jones . unresolved

I recall that this came up with a previous CL, but these dictionaries (`XRMediaQuadLayerInit`, `XRMediaCylinderLayerInit`, etc) shouldn't be optional, and it's a bug in the spec forced by our tooling that they are listed as optional there. Can we make these required? Or is there a separate blink-side bug that prevents that?

Yong Li (xWF)

Do you mean this is a bug? https://www.w3.org/TR/webxrlayers-1/#xrmediabinding
Any quick hint about how this spec gets generated? I feel it may be reasonable to be optional, because their members are all optional. So a default constructor should be fine.

Brandon Jones

The `space` is required for each of them, so they can't be optional. (It's in the base interface so it's not obvious at first glance.)

Open in Gerrit

Related details

Attention is currently required from:

• Yong Li (xWF)

Submit Requirements:

• Code-Coverage
• Code-Owners
• Code-Review
• No-Unresolved-Comments
• Review-Enforcement

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

Gerrit

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 7

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Comment-Date: Wed, 27 May 2026 18:52:22 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

Comment-In-Reply-To: Yong Li (xWF) <yyon...@google.com>

Comment-In-Reply-To: Brandon Jones <baj...@chromium.org>

[Yong Li (xWF) (Gerrit)]

Attention needed from Alexander Cooper and Brandon Jones

Yong Li (xWF) added 2 comments

File third_party/blink/renderer/modules/xr/xr_media_binding.idl

Line 12, Patchset 7: [RaisesException] XRQuadLayer createQuadLayer(HTMLVideoElement video, optional XRMediaQuadLayerInit init = {});

Brandon Jones . resolved

I recall that this came up with a previous CL, but these dictionaries (`XRMediaQuadLayerInit`, `XRMediaCylinderLayerInit`, etc) shouldn't be optional, and it's a bug in the spec forced by our tooling that they are listed as optional there. Can we make these required? Or is there a separate blink-side bug that prevents that?

Yong Li (xWF)

Do you mean this is a bug? https://www.w3.org/TR/webxrlayers-1/#xrmediabinding
Any quick hint about how this spec gets generated? I feel it may be reasonable to be optional, because their members are all optional. So a default constructor should be fine.

Brandon Jones

The `space` is required for each of them, so they can't be optional. (It's in the base interface so it's not obvious at first glance.)

Yong Li (xWF)

Done

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6928, Patchset 7: public: true,

Alexander Cooper . resolved

Status should probably be experimental; which would at least expose the regular global-interface-listing-expected.txt entry.

Yong Li (xWF)

this just defines the IDLs. It cannot be experimental yet. Do you want me to change it to experimental in the implementation CL? It always enables the feature currently.

Alexander Cooper

experimental just means it's in development.

Yong Li (xWF)

Done

Open in Gerrit

Related details

Attention is currently required from:

• Alexander Cooper
• Brandon Jones

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

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 8

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Attention: Alexander Cooper <alco...@chromium.org>

Gerrit-Comment-Date: Mon, 01 Jun 2026 17:23:40 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No

Comment-In-Reply-To: Yong Li (xWF) <yyon...@google.com>

Comment-In-Reply-To: Brandon Jones <baj...@chromium.org>

Comment-In-Reply-To: Alexander Cooper <alco...@chromium.org>

[Alexander Cooper (Gerrit)]

Attention needed from Brandon Jones and Yong Li (xWF)

Alexander Cooper voted Code-Review+1

Code-Review

+1

Open in Gerrit

Related details

Attention is currently required from:

• Brandon Jones
• Yong Li (xWF)

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

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 8

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Attention: Brandon Jones <baj...@chromium.org>

Gerrit-Comment-Date: Mon, 01 Jun 2026 18:53:54 +0000

Gerrit-HasComments: No

Gerrit-Has-Labels: Yes

[Brandon Jones (Gerrit)]

Attention needed from Alexander Cooper and Yong Li (xWF)

Brandon Jones voted Code-Review+1

Code-Review

+1

Open in Gerrit

Related details

Attention is currently required from:

• Alexander Cooper
• Yong Li (xWF)

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

Gerrit-MessageType: comment

Gerrit-Project: chromium/src

Gerrit-Branch: main

Gerrit-Change-Id: I9a4108eff089480f7f8245d7cf579619ce18a01c

Gerrit-Change-Number: 7860520

Gerrit-PatchSet: 9

Gerrit-Owner: Yong Li (xWF) <yyon...@google.com>

Gerrit-Reviewer: Alexander Cooper <alco...@chromium.org>

Gerrit-Reviewer: Brandon Jones <baj...@chromium.org>

Gerrit-Reviewer: Yong Li (xWF) <yyon...@google.com>

Gerrit-CC: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-CC: Kentaro Hara <har...@chromium.org>

Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>

Gerrit-CC: android-bu...@system.gserviceaccount.com <android-bu...@system.gserviceaccount.com>

Gerrit-CC: chromium...@chromium.org

Gerrit-Attention: Yong Li (xWF) <yyon...@google.com>

Gerrit-Attention: Alexander Cooper <alco...@chromium.org>

Gerrit-Comment-Date: Mon, 01 Jun 2026 22:48:26 +0000

Gerrit-HasComments: No

Gerrit-Has-Labels: Yes