Bug 1575983 - Remove duplicate dom.* prefs, part 2 r=njn
authorKristen Wright <kwright@mozilla.com>
Mon, 02 Sep 2019 23:35:29 +0000
changeset 491279 8e230fa660a18070977844628553f1ce5304ab99
parent 491278 5b67dc7b45744339d3a532346d5422f27a15bcde
child 491280 6b07a9592c6c211c2a38340f098d3fac2399ee92
push id114020
push userrmaries@mozilla.com
push dateTue, 03 Sep 2019 09:51:48 +0000
treeherdermozilla-inbound@08b1736b12e6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnjn
bugs1575983
milestone71.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1575983 - Remove duplicate dom.* prefs, part 2 r=njn Removes the 9 remaining dom.* duplicate prefs. Replaces default values in StaticPrefList.yaml with their all.js counterparts where needed. Imports comments from all.js. Differential Revision: https://phabricator.services.mozilla.com/D44261
modules/libpref/init/StaticPrefList.yaml
modules/libpref/init/all.js
--- a/modules/libpref/init/StaticPrefList.yaml
+++ b/modules/libpref/init/StaticPrefList.yaml
@@ -1539,19 +1539,20 @@
   value: true
   mirror: always
 
 - name: dom.ipc.cancel_content_js_when_navigating
   type: bool
   value: true
   mirror: always
 
+# Allow Flash async drawing mode in 64-bit release builds.
 - name: dom.ipc.plugins.asyncdrawing.enabled
   type: RelaxedAtomicBool
-  value: false
+  value: true
   mirror: always
 
 - name: dom.ipc.tabs.disabled
   type: bool
   value: false
   mirror: always
 
 # Process launch delay (im milliseconds).
@@ -2299,29 +2300,37 @@
   type: RelaxedAtomicBool
 #if defined(XP_WIN) || defined(XP_DARWIN) || !defined(RELEASE_OR_BETA)
   value: true
 #else
   value: false
 #endif
   mirror: always
 
+# It is often desirable to automatically start vr presentation when
+# a user puts on the VR headset.  This is done by emitting the
+# Window.vrdisplayactivate event when the headset's sensors detect it
+# being worn.  This can result in WebVR content taking over the headset
+# when the user is using it outside the browser or inadvertent start of
+# presentation due to the high sensitivity of the proximity sensor in some
+# headsets, so it is off by default.
 - name: dom.vr.autoactivate.enabled
   type: RelaxedAtomicBool
   value: false
   mirror: always
 
 # Minimum number of milliseconds that the browser will wait before
 # attempting to poll again for connected VR controllers.  The browser
 # will not attempt to poll for VR controllers until it needs to use them.
 - name: dom.vr.controller.enumerate.interval
   type: RelaxedAtomicInt32
   value: 1000
   mirror: always
 
+# The threshold value of trigger inputs for VR controllers.
 - name: dom.vr.controller_trigger_threshold
   type: AtomicFloat
   value: 0.1f
   mirror: always
 
 # Minimum number of milliseconds that the browser will wait before
 # attempting to poll again for connected VR displays.  The browser
 # will not attempt to poll for VR displays until it needs to use
@@ -2335,38 +2344,50 @@
 # frame. When content is failing to submit frames on time or the lower level
 # VR platform APIs are rejecting frames, it determines the rate at which RAF
 # callbacks will be called.
 - name: dom.vr.display.rafMaxDuration
   type: RelaxedAtomicUint32
   value: 50
   mirror: always
 
+# Minimum number of milliseconds the browser will wait before attempting
+# to re-start the VR service after an enumeration returned no devices.
 - name: dom.vr.external.notdetected.timeout
   type: RelaxedAtomicInt32
   value: 60000
   mirror: always
 
+# Minimum number of milliseconds the browser will wait before attempting
+# to re-start the VR service after a VR API (eg, OpenVR or Oculus)
+# requests that we shutdown and unload its libraries.
+# To ensure that we don't interfere with VR runtime software auto-updates,
+# we will not attempt to re-load the service until this timeout has elapsed.
 - name: dom.vr.external.quit.timeout
   type: RelaxedAtomicInt32
   value: 10000
   mirror: always
 
 # Minimum number of milliseconds that the VR session will be kept
 # alive after the browser and content no longer are using the
 # hardware.  If a VR multitasking environment, this should be set
 # very low or set to 0.
 - name: dom.vr.inactive.timeout
   type: RelaxedAtomicInt32
   value: 5000
   mirror: always
 
+# Maximum number of milliseconds the browser will wait for content to call
+# VRDisplay.requestPresent after emitting vrdisplayactivate during VR
+# link traversal.  This prevents a long running event handler for
+# vrdisplayactivate from later calling VRDisplay.requestPresent, which would
+# result in a non-responsive browser in the VR headset.
 - name: dom.vr.navigation.timeout
   type: RelaxedAtomicInt32
-  value: 1000
+  value: 5000
   mirror: always
 
 # Oculus device
 - name: dom.vr.oculus.enabled
   type: bool
 #if defined(HAVE_64BIT_BUILD) && !defined(ANDROID)
   # We are only enabling WebVR by default on 64-bit builds (Bug 1384459).
   value: true
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -239,19 +239,16 @@ pref("browser.cache.offline.capacity",  
 pref("offline-apps.quota.warn",        51200);
 
 // Don't show "Open with" option on download dialog if true.
 pref("browser.download.forbid_open_with", false);
 
 // Insecure registerProtocolHandler is disabled by default
 pref("dom.registerProtocolHandler.insecure.enabled", false);
 
-// Whether or not testing features are enabled.
-pref("dom.quotaManager.testing", false);
-
 // Whether or not indexedDB is enabled.
 pref("dom.indexedDB.enabled", true);
 // Whether or not indexedDB experimental features are enabled.
 pref("dom.indexedDB.experimental", false);
 // Enable indexedDB logging.
 pref("dom.indexedDB.logging.enabled", true);
 // Detailed output in log messages.
 pref("dom.indexedDB.logging.details", true);
@@ -305,22 +302,17 @@ pref("dom.keyboardevent.keypress.hack.us
 pref("dom.keyboardevent.keypress.hack.use_legacy_keycode_and_charcode.addl", "");
 
 // Blacklist of domains of web apps which listen for non-primary click events
 // on window global or document. The format is exactly same as
 // "dom.keyboardevent.keypress.hack.dispatch_non_printable_keys". So, check its
 // explanation for the detail.
 pref("dom.mouseevent.click.hack.use_legacy_non-primary_dispatch", "");
 
-// Whether InputEvent.dataTransfer is enabled.
-pref("dom.inputevent.datatransfer.enabled", true);
-
 #ifdef JS_BUILD_BINAST
-  pref("dom.script_loader.binast_encoding.enabled", false);
-
   // Until we're satisfied that it works nicely, we're restricting
   // BinAST to a few partner sites:
   // - A subset of Facebook
   // - A subset of Cloudflare
   pref("dom.script_loader.binast_encoding.domain.restrict.list", "*.facebook.com,static.xx.fbcdn.net,*.cloudflare.com,*.cloudflarestream.com,unpkg.com");
 #endif
 
 // Fastback caching - if this pref is negative, then we calculate the number
@@ -2937,18 +2929,16 @@ pref("dom.ipc.plugins.flash.disable-prot
 
 pref("dom.ipc.plugins.flash.subprocess.crashreporter.enabled", true);
 pref("dom.ipc.plugins.reportCrashURL", true);
 
 // How long we wait before unloading an idle plugin process.
 // Defaults to 30 seconds.
 pref("dom.ipc.plugins.unloadTimeoutSecs", 30);
 
-// Allow Flash async drawing mode in 64-bit release builds
-pref("dom.ipc.plugins.asyncdrawing.enabled", true);
 // Force the accelerated direct path for a subset of Flash wmode values
 pref("dom.ipc.plugins.forcedirect.enabled", true);
 
 // Enable multi by default.
 #if !defined(MOZ_ASAN)
   pref("dom.ipc.processCount", 8);
 #else
   pref("dom.ipc.processCount", 4);
@@ -4694,41 +4684,16 @@ pref("memory.blob_report.stack_frames", 
 pref("io.activity.enabled", false);
 
 // If true, reuse the same global for (almost) everything loaded by the component
 // loader (JS components, JSMs, etc). This saves memory, but makes it possible
 // for the scripts to interfere with each other.  A restart is required for this
 // to take effect.
 pref("jsloader.shareGlobal", true);
 
-// It is often desirable to automatically start vr presentation when
-// a user puts on the VR headset.  This is done by emitting the
-// Window.vrdisplayactivate event when the headset's sensors detect it
-// being worn.  This can result in WebVR content taking over the headset
-// when the user is using it outside the browser or inadvertent start of
-// presentation due to the high sensitivity of the proximity sensor in some
-// headsets, so it is off by default.
-pref("dom.vr.autoactivate.enabled", false);
-// The threshold value of trigger inputs for VR controllers
-pref("dom.vr.controller_trigger_threshold", "0.1");
-// Minimum number of milliseconds the browser will wait before attempting
-// to re-start the VR service after an enumeration returned no devices.
-pref("dom.vr.external.notdetected.timeout", 60000);
-// Minimum number of milliseconds the browser will wait before attempting
-// to re-start the VR service after a VR API (eg, OpenVR or Oculus)
-// requests that we shutdown and unload its libraries.
-// To ensure that we don't interfere with VR runtime software auto-updates,
-// we will not attempt to re-load the service until this timeout has elapsed.
-pref("dom.vr.external.quit.timeout", 10000);
-// Maximum number of milliseconds the browser will wait for content to call
-// VRDisplay.requestPresent after emitting vrdisplayactivate during VR
-// link traversal.  This prevents a long running event handler for
-// vrdisplayactivate from later calling VRDisplay.requestPresent, which would
-// result in a non-responsive browser in the VR headset.
-pref("dom.vr.navigation.timeout", 5000);
 // path to OSVR DLLs
 pref("gfx.vr.osvr.utilLibPath", "");
 pref("gfx.vr.osvr.commonLibPath", "");
 pref("gfx.vr.osvr.clientLibPath", "");
 pref("gfx.vr.osvr.clientKitLibPath", "");
 
 // nsMemoryInfoDumper can watch a fifo in the temp directory and take various
 // actions when the fifo is written to.  Disable this in general.