[CLOSED] Changed requestPresent to accept a sequence of VRLayers

[cvan]

Issue by toji
Thursday Mar 31, 2016 at 17:29 GMT
Originally opened as MozillaReality/webvr-spec#28

---

Also added maxLayers capability. As per conversations with @vvuk. Still restricts the v1 spec to only accepting a single layer, but makes future changes to allows for more complex compositing more sensible.

---

toji included the following code: https://github.com/MozVR/webvr-spec/pull/28/commits

[cvan]

Comment by toji
Thursday Mar 31, 2016 at 17:55 GMT

---

If this change were to go into effect I would probably keep the non-array variant of the function in Chrome's experimental builds for a bit and have them print a deprecation warning to the console when used, just so any 1.0 compatible content people have built doesn't immediately fail.

[cvan]

Comment by cvan
Friday Apr 01, 2016 at 01:39 GMT

---

needs rebase

[cvan]

Comment by toji
Saturday Apr 02, 2016 at 00:01 GMT

---

Rebased and addressed @cvan's comments. Thanks!

[cvan]

Comment by cvan
Saturday Apr 02, 2016 at 00:47 GMT

---

this looks great. r+ and that's no joke 👍

[cvan]

Comment by mkeblx
Thursday Apr 14, 2016 at 21:48 GMT

---

Saw this in IDL.
https://github.com/MozVR/webvr-spec/blob/master/index.idl#L44

[cvan]

Comment by cvan
Thursday Apr 14, 2016 at 23:37 GMT

---

oh, good catch. gotta fix that.