Bug 1320343 - Avoid to set Preference in the GPU process; r?kip draft
authorDaosheng Mu <daoshengmu@gmail.com>
Sat, 26 Nov 2016 23:06:34 +0800
changeset 444170 8b001bf16101647d52a1b7a49166787d6cdf1577
parent 444155 26773c7afa55b6f8ad725d5a00850b379eb0a10f
child 538248 73458a1ea0f8209326039eb8c9217048f83ce616
push id37213
push userbmo:dmu@mozilla.com
push dateSat, 26 Nov 2016 16:02:41 +0000
reviewerskip
bugs1320343
milestone53.0a1
Bug 1320343 - Avoid to set Preference in the GPU process; r?kip MozReview-Commit-ID: I73foOQh7L0
gfx/vr/VRManager.cpp
--- a/gfx/vr/VRManager.cpp
+++ b/gfx/vr/VRManager.cpp
@@ -90,17 +90,18 @@ VRManager::VRManager()
 
   // OSVR is cross platform compatible
   mgr = VRDisplayManagerOSVR::Create();
   if (mgr) {
       mManagers.AppendElement(mgr);
   }
 #endif
   // Enable gamepad extensions while VR is enabled.
-  if (gfxPrefs::VREnabled()) {
+  // Preference only can be set at the Parent process.
+  if (XRE_IsParentProcess() && gfxPrefs::VREnabled()) {
     Preferences::SetBool("dom.gamepad.extensions.enabled", true);
   }
 }
 
 VRManager::~VRManager()
 {
   MOZ_ASSERT(NS_IsMainThread());
   MOZ_ASSERT(!mInitialized);