Three.js: WebXRManager: renderer.getContext() throws

[avaer]

It looks like WebXRManager is constructed in WebGLRenderer, and tries to getContext() of the renderer.

However, this happens before the getContext accessor is set up in the first place.

Should WebXRManager initialization be moved down?

[Mugen87]

Should WebXRManager initialization be moved down?

That's probably the best solution. Do you want to make a PR for this? 😇

[mrdoob]

I think I would solve it differently.

[mrdoob]

Thanks for the bug report!

[fernandojsg]

Yeah nice catch!

[takahirox]

Just my two cents. I know it may be just my preference but I prefer moving down the manager initialization (or something) because (new) users might think having both renderer and gl arguments in the constructor looks redundant.