From f1e46d8145c834a712f6be7c1e5bd590c72da749 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Tue, 28 Apr 2015 07:49:13 +0100
Subject: Only grab focus after link transition.

---
 background_scripts/main.coffee         | 2 +-
 content_scripts/vimium_frontend.coffee | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/background_scripts/main.coffee b/background_scripts/main.coffee
index e782a217..e3d1d214 100644
--- a/background_scripts/main.coffee
+++ b/background_scripts/main.coffee
@@ -100,7 +100,7 @@ root.isEnabledForUrl = isEnabledForUrl = (request, sender) ->
   }
 
 onURLChange = (details) ->
-  chrome.tabs.sendMessage details.tabId, name: "checkEnabledAfterURLChange"
+  chrome.tabs.sendMessage details.tabId, extend details, name: "checkEnabledAfterURLChange"
 
 # Re-check whether Vimium is enabled for a frame when the url changes without a reload.
 chrome.webNavigation.onHistoryStateUpdated.addListener onURLChange # history.pushState.
diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index 28102527..14891707 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -608,11 +608,11 @@ checkIfEnabledForUrl = (frameIsFocused = windowIsFocused()) ->
 
 # When we're informed by the background page that a URL in this tab has changed, we check if we have the
 # correct enabled state (but only if this frame has the focus).
-checkEnabledAfterURLChange = ->
+checkEnabledAfterURLChange = (request) ->
   if windowIsFocused()
     checkIfEnabledForUrl()
     # We also grab back the focus.  See #1588.
-    new GrabBackFocus
+    new GrabBackFocus() if request.transitionType in [ "link", "form_submit" ]
 
 # Exported to window, but only for DOM tests.
 window.refreshCompletionKeys = (response) ->
-- 
cgit v1.2.3