Bug 1539470 - Prevent tabs opened in the background from affecting the CFR Pin Tab visits counter

Author: piatra

lib/ASRouterTriggerListeners.jsm

@@ -30,6 +30,16 @@ async function checkStartupFinished(win) {
   }
 }
 
+function isFocusedTab(aBrowser, aLocationURI) {
+  try {
+    if (aBrowser.ownerGlobal.gBrowser.currentURI.spec === aLocationURI.aLocationURI.spec) {
+      return true;
+    }
+  } catch (e) {} // nsIURI.host can throw for non-nsStandardURL nsIURIs
+
+  return false;
+}
+
 function isPrivateWindow(win) {
   return !(win instanceof Ci.nsIDOMWindow) || win.closed || PrivateBrowsingUtils.isWindowPrivate(win);
 }
@@ -179,6 +189,12 @@ this.ASRouterTriggerListeners = new Map([
     },
 
     onLocationChange(aBrowser, aWebProgress, aRequest, aLocationURI, aFlags) {
+      // If a new tab was opened in the background we don't want it to increment
+      // the visits count. That will happen anyway because of the `onTabSwitch`
+      // listener that fires when the tab is selected.
+      if (!isFocusedTab(aBrowser, aLocationURI)) {
+        return;
+      }
       // Some websites trigger redirect events after they finish loading even
       // though the location remains the same. This results in onLocationChange
       // events to be fired twice.