From c6c278f9d5f1b3f35ae6c0e79cca3a94aa2660a2 Mon Sep 17 00:00:00 2001
From: Juanjo Diaz <juanjo.diazmo@gmail.com>
Date: Wed, 27 Feb 2019 10:14:50 +0200
Subject: [PATCH] Replace unnecessary function supportsCursorURIs by a constant
 variable

---
 kasmweb/core/util/browser.js | 36 +++++++++++++++---------------------
 kasmweb/core/util/cursor.js  |  2 +-
 2 files changed, 16 insertions(+), 22 deletions(-)

diff --git a/kasmweb/core/util/browser.js b/kasmweb/core/util/browser.js
index 78e104d..8996cfe 100644
--- a/kasmweb/core/util/browser.js
+++ b/kasmweb/core/util/browser.js
@@ -25,30 +25,24 @@ window.addEventListener('touchstart', function onFirstTouch() {
 // brings us a bit closer but is not optimal.
 export let dragThreshold = 10 * (window.devicePixelRatio || 1);
 
-let _cursor_uris_supported = null;
-
-export function supportsCursorURIs() {
-    if (_cursor_uris_supported === null) {
-        try {
-            const target = document.createElement('canvas');
-            target.style.cursor = 'url("data:image/x-icon;base64,AAACAAEACAgAAAIAAgA4AQAAFgAAACgAAAAIAAAAEAAAAAEAIAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAAAAD/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////AAAAAAAAAAAAAAAAAAAAAA==") 2 2, default';
-
-            if (target.style.cursor) {
-                Log.Info("Data URI scheme cursor supported");
-                _cursor_uris_supported = true;
-            } else {
-                Log.Warn("Data URI scheme cursor not supported");
-                _cursor_uris_supported = false;
-            }
-        } catch (exc) {
-            Log.Error("Data URI scheme cursor test exception: " + exc);
-            _cursor_uris_supported = false;
-        }
-    }
+let _supportsCursorURIs = false;
 
-    return _cursor_uris_supported;
+try {
+    const target = document.createElement('canvas');
+    target.style.cursor = 'url("data:image/x-icon;base64,AAACAAEACAgAAAIAAgA4AQAAFgAAACgAAAAIAAAAEAAAAAEAIAAAAAAAEAAAAAAAAAAAAAAAAAAAAAAAAAD/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////AAAAAAAAAAAAAAAAAAAAAA==") 2 2, default';
+
+    if (target.style.cursor) {
+        Log.Info("Data URI scheme cursor supported");
+        _supportsCursorURIs = true;
+    } else {
+        Log.Warn("Data URI scheme cursor not supported");
+    }
+} catch (exc) {
+    Log.Error("Data URI scheme cursor test exception: " + exc);
 }
 
+export const supportsCursorURIs = _supportsCursorURIs;
+
 let _supportsImageMetadata = false;
 try {
     new ImageData(new Uint8ClampedArray(4), 1, 1);
diff --git a/kasmweb/core/util/cursor.js b/kasmweb/core/util/cursor.js
index b73f862..0d0b754 100644
--- a/kasmweb/core/util/cursor.js
+++ b/kasmweb/core/util/cursor.js
@@ -6,7 +6,7 @@
 
 import { supportsCursorURIs, isTouchDevice } from './browser.js';
 
-const useFallback = !supportsCursorURIs() || isTouchDevice;
+const useFallback = !supportsCursorURIs || isTouchDevice;
 
 export default class Cursor {
     constructor() {