throttle updateControllerList while keeping getGamepads call (#3112)

aframevr · Nov 7, 2017 · 7cc2cef · 7cc2cef
1 parent 3a933b4
commit 7cc2cef
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/systems/tracked-controls.js b/src/systems/tracked-controls.js
@@ -1,4 +1,5 @@
 var registerSystem = require('../core/system').registerSystem;
+var utils = require('../utils');
 
 /**
  * Tracked controls system.
@@ -10,7 +11,8 @@ module.exports.System = registerSystem('tracked-controls', {
 
     this.controllers = [];
 
-    this.updateControllerList();
+    this.updateControllerList(navigator.getGamepads && navigator.getGamepads());
+    this.throttledUpdateControllerList = utils.throttle(this.updateControllerList, 500, this);
 
     if (!navigator.getVRDisplays) { return; }
 
@@ -22,20 +24,21 @@ module.exports.System = registerSystem('tracked-controls', {
   },
 
   tick: function () {
-    this.updateControllerList();
+    var gamepads;
+    // Call getGamepads for Chrome.
+    gamepads = navigator.getGamepads && navigator.getGamepads();
+    this.throttledUpdateControllerList(gamepads);
   },
 
   /**
    * Update controller list.
    */
-  updateControllerList: function () {
+  updateControllerList: function (gamepads) {
     var controllers = this.controllers;
     var gamepad;
-    var gamepads;
     var i;
     var prevCount;
 
-    gamepads = navigator.getGamepads && navigator.getGamepads();
     if (!gamepads) { return; }
 
     prevCount = controllers.length;