Fix #451, close the search tab after an hour if it's unused

mozilla-extensions · Oct 23, 2019 · db5ad1e · db5ad1e
1 parent 2766795
commit db5ad1e
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/extension/intents/search/search.js b/extension/intents/search/search.js
@@ -2,13 +2,23 @@
 
 this.intents.search = (function() {
   const exports = {};
+  // Close the search tab after this amount of time:
+  const CLOSE_TIME = 1000 * 60 * 60; // 1 hour
+  let closeTabTimeout = null;
   let _searchTabId;
   const START_URL = "https://www.google.com";
   let lastSearchInfo;
   let lastSearchIndex;
   let lastTabId;
 
   async function openSearchTab() {
+    if (closeTabTimeout) {
+      clearTimeout(closeTabTimeout);
+      closeTabTimeout = null;
+    }
+    closeTabTimeout = setTimeout(() => {
+      closeSearchTab();
+    }, CLOSE_TIME);
     if (_searchTabId) {
       try {
         await browser.tabs.get(_searchTabId);
@@ -36,6 +46,15 @@ this.intents.search = (function() {
     return _searchTabId;
   }
 
+  async function closeSearchTab() {
+    if (!_searchTabId) {
+      return;
+    }
+    const tabId = _searchTabId;
+    _searchTabId = null;
+    await browser.tabs.remove(tabId);
+  }
+
   async function performSearch(query) {
     const tabId = await openSearchTab();
     const url = searching.googleSearchUrl(query) + "&voice";