From 7d69ecc00595c936efee2561ea73bd1dfcf07395 Mon Sep 17 00:00:00 2001
From: griffpatch <griffpatch@users.noreply.github.com>
Date: Wed, 8 Feb 2017 08:11:10 +0000
Subject: [PATCH] Optimisation - Only check browser compatability once

Disable use of more accurate timer temporarilly due to performance.
However, with the rearrange and introduction of the 'nowObj' it turns
out most of the delay is in resolving 'self.performance'... so keeping
this cached in nowObj actually mitigates most of the delay.
---
 src/util/timer.js | 50 +++++++++++++++++++++++++++++------------------
 1 file changed, 31 insertions(+), 19 deletions(-)

diff --git a/src/util/timer.js b/src/util/timer.js
index a0b620112..9a60e5688 100644
--- a/src/util/timer.js
+++ b/src/util/timer.js
@@ -23,16 +23,36 @@ var Timer = function () {};
  */
 Timer.prototype.startTime = 0;
 
+/**
+ * Disable use of self.performance for now as it results in lower performance
+ * However, instancing it like below (caching the self.performance to a local variable) negates most of the issues.
+ * @type {boolean}
+ */
+var USE_PERFORMANCE = false;
+
+/**
+ * Legacy object to allow for us to call now to get the old style date time (for backwards compatibility)
+ * @deprecated This is only called via the nowObj.now() if no other means is possible...
+ */
+var legacyDateCode = {
+    now: function () {
+        return new Date().getTime();
+    }
+};
+
+/**
+ * Use this object to route all time functions through single access points.
+ */
+var nowObj = (USE_PERFORMANCE && typeof self !== 'undefined' && self.performance && 'now' in self.performance) ?
+    self.performance : Date.now ? Date : legacyDateCode;
+
 /**
  * Return the currently known absolute time, in ms precision.
  * @returns {number} ms elapsed since 1 January 1970 00:00:00 UTC.
  */
-Timer.prototype.time = Date.now ?
-    function () {
-        return Date.now();
-    } : function () {
-        return new Date().getTime();
-    };
+Timer.prototype.time = function () {
+    return nowObj.now();
+};
 
 /**
  * Returns a time accurate relative to other times produced by this function.
@@ -41,28 +61,20 @@ Timer.prototype.time = Date.now ?
  * Not guaranteed to produce the same absolute values per-system.
  * @returns {number} ms-scale accurate time relative to other relative times.
  */
-Timer.prototype.relativeTime =
-    (typeof self !== 'undefined' && self.performance && 'now' in self.performance) ?
-        function () {
-            return self.performance.now();
-        } : function () {
-            return this.time();
-        };
+Timer.prototype.relativeTime = function () {
+    return nowObj.now();
+};
 
 /**
  * Start a timer for measuring elapsed time,
  * at the most accurate precision possible.
  */
 Timer.prototype.start = function () {
-    this.startTime = this.relativeTime();
+    this.startTime = nowObj.now();
 };
 
-/**
- * Check time elapsed since `timer.start` was called.
- * @returns {number} Time elapsed, in ms (possibly sub-ms precision).
- */
 Timer.prototype.timeElapsed = function () {
-    return this.relativeTime() - this.startTime;
+    return nowObj.now() - this.startTime;
 };
 
 module.exports = Timer;