Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XRDOMOverlayState should be an interface #41

Open
foolip opened this issue Jan 29, 2021 · 0 comments
Open

XRDOMOverlayState should be an interface #41

foolip opened this issue Jan 29, 2021 · 0 comments

Comments

@foolip
Copy link

foolip commented Jan 29, 2021

Per the spec, XRDOMOverlayState should be a dictionary, but in Chromium it's an interface.

Because XRDOMOverlayState is the type of an attribute on an interface, it doesn't really work to use a dictionary, since dictionaries are passed by value, and one would end up returning a new object each time, such that xrSession.domOverlay == xrSession.domOverlay would not hold.

Web IDL says simply:

Dictionaries must not be used as the type of an attribute or constant.

This doesn't make much of a difference, but will mean that there will be an window.XRDOMOverlayState interface object. (There is in Chromium, but shouldn't be per spec, which is how I noticed.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant