fire VRDisplay connect only when polyfilled display is used. Fixes #303

immersive-web · Mar 14, 2018 · 73e6bb0 · 73e6bb0
1 parent 2c67610
commit 73e6bb0
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/examples/index.html b/examples/index.html
@@ -232,9 +232,14 @@
   buttons.hidden = vrDisplay.isPresenting;
 }
 
+function onVRDisplayConnect(e) {
+  console.log('onVRDisplayConnect', (e.display || (e.detail && e.detail.display)));
+}
+
 // Resize the WebGL canvas when we resize and also when we change modes.
 window.addEventListener('resize', onResize);
 window.addEventListener('vrdisplaypresentchange', onVRDisplayPresentChange);
+window.addEventListener('vrdisplayconnect', onVRDisplayConnect);
 
 // Button click handlers.
 document.querySelector('button#fullscreen').addEventListener('click', function() {

diff --git a/src/webvr-polyfill.js b/src/webvr-polyfill.js
@@ -39,6 +39,13 @@ function WebVRPolyfill(config) {
   // inject our own polyfill
   if (!this.hasNative || this.config.PROVIDE_MOBILE_VRDISPLAY && isMobile()) {
     this.enable();
+    // If we need to create a CardboardVRDisplay, fire the `vrdisplayconnect`
+    // event when the polyfill is enabled
+    this.getVRDisplays().then(function (displays) {
+      if (displays && displays[0] && displays[0].fireVRDisplayConnect_) {
+        displays[0].fireVRDisplayConnect_();
+      }
+    });
   }
 }