getVRStatus feels slightly subjective, suggesting we use inVR instead…

… to query if we're in VR

Summary: Suggested function name change for discussion

Reviewed By: mikearmstrong001

Differential Revision: D5103570

fbshipit-source-id: f368271

Libraries/Utilities/VrHeadModel.js

@@ -24,18 +24,18 @@ class VrHeadModelImpl {
   constructor() {
     this.headMatrix = MatrixMath.createIdentityMatrix();
     this.viewMatrix = MatrixMath.createIdentityMatrix();
-    this.inVR = false;
+    this._inVR = false;
     this.fov = 0;
     this.aspect = 1;
     this._headMatrixListener = RCTDeviceEventEmitter.addListener(
       'onReceivedHeadMatrix',
       this._onReceivedHeadMatrix.bind(this)
     );
     RCTDeviceEventEmitter.addListener('onEnterVR', () => {
-      this.inVR = true;
+      this._inVR = true;
     });
     RCTDeviceEventEmitter.addListener('onExitVR', () => {
-      this.inVR = false;
+      this._inVR = false;
     });
   }
 
@@ -165,7 +165,15 @@ class VrHeadModelImpl {
    * Return true if the headset is currently used
    */
   getVRStatus() {
-    return this.inVR;
+    console.warn('getVRStatus is deprecated.  Please use inVR instead');
+    return this._inVR;
+  }
+
+  /**
+   * Return true if using a VR headset, false if viewing outside of VR
+   */
+  inVR() {
+    return this._inVR;
   }
 }