[Pkg-mozext-commits] [tabmixplus] 19/61: Workaround for bug 1157404 - [e10s] Possible to end up with two visuallyselected="true" tabs at the same time

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository tabmixplus.

commit 5256b4184c623ca999c3cfe904d2a72e5d4236c0
Author: onemen <tabmix.onemen at gmail.com>
Date:   Thu Jul 16 15:10:15 2015 +0300

    Workaround for bug 1157404 - [e10s] Possible to end up with two visuallyselected="true" tabs at the same time
---
 chrome/content/links/userInterface.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/chrome/content/links/userInterface.js b/chrome/content/links/userInterface.js
index 26853f4..a9abb7c 100644
--- a/chrome/content/links/userInterface.js
+++ b/chrome/content/links/userInterface.js
@@ -172,6 +172,13 @@ function TMP_BrowserOpenTab(aTab, replaceLastTab) {
      }
    }
 
+   // make sure to update recently used tabs
+   // if user open many tabs quickly select event don't have time to fire
+   // before new tab select
+   if (!loadInBackground) {
+      gBrowser.selectedTab = newTab;
+      TMP_LastTab.PushSelectedTab();
+   }
    if (aTab && aTab.localName == "tab")
       gBrowser.moveTabTo(newTab, aTab._tPos + 1);
    else if (!replaceLastTab && Tabmix.prefs.getBoolPref("openNewTabNext")) {
@@ -179,13 +186,6 @@ function TMP_BrowserOpenTab(aTab, replaceLastTab) {
       // and it mess with recently used tabs order
       gBrowser.moveTabTo(newTab, selectedTab._tPos + 1);
    }
-   // make sure to update recently used tabs
-   // if user open many tabs quickly select event don't have time to fire
-   // before new tab select
-   if (!loadInBackground) {
-     gBrowser.selectedTab = newTab;
-     TMP_LastTab.PushSelectedTab();
-   }
 
    gBrowser.selectedBrowser.focus();
    // focus the address bar on new tab

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/tabmixplus.git