Bug 1750902 - Disable dom.vr.enabled by default. r=jrmuizel,jmathies,emilio

Preserve testing, just disable the pref by default.

Differential Revision: https://phabricator.services.mozilla.com/D136411

dom/tests/mochitest/general/test_interfaces.js

@@ -149,6 +149,7 @@ var legacyMozPrefixedInterfaces = [
 //            except to remove items from it!
 
 // IMPORTANT: Do not change the list below without review from a DOM peer!
+// (You can request review on Phabricator via r=#webidl)
 var interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "AbortController", insecureContext: true },
@@ -1286,40 +1287,6 @@ var interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "VisualViewport", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "VRDisplay", releaseNonWindowsAndMac: false },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VRDisplayCapabilities",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VRDisplayEvent",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VREyeParameters",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VRFieldOfView",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VRFrameData",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "VRPose", releaseNonWindowsAndMac: false },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
-  {
-    name: "VRStageParameters",
-    releaseNonWindowsAndMac: false,
-  },
-  // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "VTTCue", insecureContext: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   { name: "VTTRegion", insecureContext: true },
@@ -1411,7 +1378,6 @@ function entryDisabled(entry) {
     entry.fennecOrDesktop === (isAndroid && !isFennec) ||
     entry.fennec === !isFennec ||
     entry.release === !isRelease ||
-    entry.releaseNonWindowsAndMac === !(isRelease && !isWindows && !isMac) ||
     entry.releaseNonWindows === !(isRelease && !isWindows) ||
     // The insecureContext test is very purposefully converting
     // entry.insecureContext to boolean, so undefined will convert to

dom/vr/test/crashtests/crashtests.list

@@ -1 +1 @@
-pref(dom.vr.always_support_vr,true) load enumerate_vr_on_dying_window.html
+pref(dom.vr.enabled,true) pref(dom.vr.always_support_vr,true) load enumerate_vr_on_dying_window.html

dom/vr/test/mochitest/runVRTest.js

@@ -2,6 +2,7 @@ function runVRTest(callback) {
   SpecialPowers.pushPrefEnv(
     {
       set: [
+        ["dom.vr.enabled", true],
         ["dom.vr.puppet.enabled", true],
         ["dom.vr.require-gesture", false],
         ["dom.vr.test.enabled", true],

dom/vr/test/reftest/reftest.list

@@ -1,6 +1,6 @@
 # WebVR Reftests
 # Please confirm there is no other VR display connected. Otherwise, VRPuppetDisplay can't be attached.
-defaults pref(dom.vr.puppet.enabled,true) pref(dom.vr.test.enabled,true) pref(dom.vr.require-gesture,false) pref(dom.vr.puppet.submitframe,1) pref(dom.vr.display.rafMaxDuration,200) pref(dom.vr.display.enumerate.interval,0) pref(dom.vr.controller.enumerate.interval,0)
+defaults pref(dom.vr.enabled,true) pref(dom.vr.puppet.enabled,true) pref(dom.vr.test.enabled,true) pref(dom.vr.require-gesture,false) pref(dom.vr.puppet.submitframe,1) pref(dom.vr.display.rafMaxDuration,200) pref(dom.vr.display.enumerate.interval,0) pref(dom.vr.controller.enumerate.interval,0)
 # WebVR Tests have been disabled as refactoring of gfxVRPuppet is landing.  Dependencies for re-enabling these are tracked by meta bug 1555185.
 # VR SubmitFrame is only implemented for D3D11.1 and MacOSX now.
 # Our Windows 7 test machines don't support D3D11.1, so we run these tests on Windows 8+ only.

modules/libpref/init/StaticPrefList.yaml

@@ -3746,15 +3746,10 @@
   mirror: always
 
 # Is support for WebVR APIs enabled?
-# Enabled by default in beta and release for Windows and OS X and for all
-# platforms in nightly and aurora.
+# Disabled everywhere, but not removed.
 - name: dom.vr.enabled
   type: RelaxedAtomicBool
-#if defined(XP_WIN) || defined(XP_DARWIN) || !defined(RELEASE_OR_BETA)
-  value: true
-#else
   value: false
-#endif
   mirror: always
 
 # Should VR sessions always be reported as supported, without first