From 226cfb8011e83e48b9c723ebc891ad0e9a7c82ec Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Fri, 19 Dec 2014 05:38:29 +0000
Subject: Tidy cursor hider, move haveChromeVersion to utils.

---
 content_scripts/vimium_frontend.coffee | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

(limited to 'content_scripts')

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index 6f099e54..21a05de6 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -1107,9 +1107,7 @@ Tween =
 
 CursorHider =
   #
-  # Hides the cursor when the browser scrolls, and prevent mouse from hovering while invisible.
-  # Disabled for Chrome versions less than 39.0.2171.71 due to a browser error.
-  # NOTE(smblott) onScroll and onMouseMove events come in pairs.
+  # Hide the cursor when the browser scrolls, and prevent mouse from hovering while invisible.
   #
   cursorHideStyle: null
   isScrolling: false
@@ -1125,9 +1123,9 @@ CursorHider =
     CursorHider.isScrolling = false
 
   init: ->
-    # Disable for Chrome versions less than 39.0.2171.71 due to a browser error.
-    chromeVersion = navigator.appVersion.match(/Chrome\/(.*?) /)?[1] || ""
-    return if 0 <= Utils.compareVersions "39.0.2171.71", chromeVersion
+    # Disable cursor hiding for Chrome versions less than 39.0.2171.71 due to a suspected browser error.
+    # See #1345 and #1348.
+    return unless Utils.haveChromeVersion "39.0.2171.71"
 
     @cursorHideStyle = document.createElement("style")
     @cursorHideStyle.innerHTML = """
-- 
cgit v1.2.3