From f2d7e185c704e3cf8f6ca0c9d3d4da2498610475 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Thu, 14 Dec 2017 14:08:35 +0000
Subject: Detect click listeners for link hints.

This is a simpler version of #1167.  It detects clickable elements with
listeners added with `addEventListener()`.

It includes some of @mrmr1993's ideas from #1167 (in fact, it's mostly
those ideas tweaked into a slightly different form).
---
 content_scripts/injected.coffee   | 20 ++++++++++++++++++++
 content_scripts/link_hints.coffee |  3 +++
 2 files changed, 23 insertions(+)
 create mode 100644 content_scripts/injected.coffee

(limited to 'content_scripts')

diff --git a/content_scripts/injected.coffee b/content_scripts/injected.coffee
new file mode 100644
index 00000000..d833d09c
--- /dev/null
+++ b/content_scripts/injected.coffee
@@ -0,0 +1,20 @@
+# The code in `injectedCode()`, below, is injected into the page's own execution context.
+#
+# This is based on method 2b here: http://stackoverflow.com/a/9517879, and
+# @mrmr1993's #1167.
+
+window.vimiumOnClickAttributeName = "_vimium-has-onclick-listener"
+
+injectedCode = (vimiumOnClickAttributeName) ->
+  # Note the presence of "click" listeners installed with `addEventListener()` (for link hints).
+  _addEventListener = Element::addEventListener
+
+  Element::addEventListener = (type, listener, useCapture) ->
+    @setAttribute vimiumOnClickAttributeName, "" if type == "click"
+    _addEventListener.apply this, arguments
+
+script = document.createElement "script"
+script.textContent = "(#{injectedCode.toString()})('#{vimiumOnClickAttributeName}')"
+(document.head || document.documentElement).appendChild script
+script.remove()
+
diff --git a/content_scripts/link_hints.coffee b/content_scripts/link_hints.coffee
index 0592c96d..e5c6b3ea 100644
--- a/content_scripts/link_hints.coffee
+++ b/content_scripts/link_hints.coffee
@@ -706,6 +706,9 @@ LocalHints =
         isClickable = true
         reason = "Open."
 
+    # Detect elements with "click" listeners installed with `addEventListener()`.
+    isClickable ||= element.hasAttribute vimiumOnClickAttributeName
+
     # An element with a class name containing the text "button" might be clickable.  However, real clickables
     # are often wrapped in elements with such class names.  So, when we find clickables based only on their
     # class name, we mark them as unreliable.
-- 
cgit v1.2.3