From 5dcff0bef00eb2deccda0c22fa911ead576aa0dc Mon Sep 17 00:00:00 2001
From: S C P <26688071+s-c-p@users.noreply.github.com>
Date: Sat, 11 May 2024 09:24:33 +0530
Subject: [PATCH] Illustrative function name

updateActiveTab has ambiguous meaning, most prominent of which doesn't mean what we are actually trying to do.
---
 bookmark-it/background.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/bookmark-it/background.js b/bookmark-it/background.js
index 8e76f2bd..b4d73e68 100644
--- a/bookmark-it/background.js
+++ b/bookmark-it/background.js
@@ -39,7 +39,7 @@ browser.browserAction.onClicked.addListener(toggleBookmark);
 /*
  * Switches currentTab and currentBookmark to reflect the currently active tab
  */
-function updateActiveTab(tabs) {
+function updateAddonStateForActiveTab(tabs) {
 
   function isSupportedProtocol(urlString) {
     let supportedProtocols = ["https:", "http:", "ftp:", "file:"];
@@ -68,19 +68,19 @@ function updateActiveTab(tabs) {
 }
 
 // listen for bookmarks being created
-browser.bookmarks.onCreated.addListener(updateActiveTab);
+browser.bookmarks.onCreated.addListener(updateAddonStateForActiveTab);
 
 // listen for bookmarks being removed
-browser.bookmarks.onRemoved.addListener(updateActiveTab);
+browser.bookmarks.onRemoved.addListener(updateAddonStateForActiveTab);
 
 // listen to tab URL changes
-browser.tabs.onUpdated.addListener(updateActiveTab);
+browser.tabs.onUpdated.addListener(updateAddonStateForActiveTab);
 
 // listen to tab switching
-browser.tabs.onActivated.addListener(updateActiveTab);
+browser.tabs.onActivated.addListener(updateAddonStateForActiveTab);
 
 // listen for window switching
-browser.windows.onFocusChanged.addListener(updateActiveTab);
+browser.windows.onFocusChanged.addListener(updateAddonStateForActiveTab);
 
 // update when the extension loads initially
-updateActiveTab();
+updateAddonStateForActiveTab();