bubbleEvent should manage the native event propagation.

If the handlers do not want it bubbled up the internal stack, we can
safely assume that they do not want it bubbled up the native stack as
well.

This fixes some problems e.g. with Google search pages.

1 files changed, 3 insertions, 1 deletions

diff --git a/vimiumFrontend.js b/vimiumFrontend.js
index 6f02bb1c..283f8d08 100644
--- a/vimiumFrontend.js
+++ b/vimiumFrontend.js
@@ -401,8 +401,10 @@ function bubbleEvent(type, event) {
   for (var i = handlerStack.length-1; i >= 0; i--) {
     // We need to check for existence of handler because the last function call may have caused the release of
     // more than one handler.
-    if (handlerStack[i] && handlerStack[i][type] && !handlerStack[i][type](event))
+    if (handlerStack[i] && handlerStack[i][type] && !handlerStack[i][type](event)) {
+      suppressEvent(event);
       return false;
+    }
   }
   return true;
 }