Upstream monkey patches

[annevk]

It seems this document modifies the HTML Standard, e.g., the getContext() method. This should be defined there.

[annevk]

Upon a closer look perhaps it's WebGL 1/2 that are being patched?

[Manishearth]

Somewhat both, getContext() gets a new flag but it mostly affects context creation from the webgl spec

[Manishearth]

So the issue here is that WebGL patches itself when talking about attributes too. The context creation algorithm does not really look at attributes, instead the attributes are defined by their impact on context creation.

We could potentially use language similar to theirs -- instead of patching an algorithm, we can say that "this attribute when set makes the context creation do xyz"

[Manishearth]

We can also ask them to mention this flag upstream with a "full semantics defined in the webxr spec"

[annevk]

Yeah, I think there needs to be a path from HTML to XR, but going through WebGL is reasonable as there's nothing changed in HTML directly. How the dictionary passed as second argument is handled is entirely up to WebGL at the moment as HTML takes any.

[annevk]

I'm not really sure how that PR addressed my concern?

[Manishearth]

@annevk It defined these attributes the same way as the webgl spec itself defines similar attributes. We do hope to eventually move this into that spec, but until then this is the best we can do.

[annevk]

You could file an upstream issue and point to that from the specification?

[Manishearth]

Yeah, I think @toji is actively discussing this stuff with the WG though, I don't know the full status of that. Maybe he can file the issue?