Merge branch 'tabslideshow'

src/excmds.ts

@@ -2168,6 +2168,52 @@ export async function tabopen(...addressarr: string[]) {
     })
 }
 
+/**
+  Passes its first argument to `tabopen -b`. Once the tab opened by `tabopen
+  -b` is activated/selected/focused, opens its second argument with `tabopen
+  -b`. Once the second tab is activated/selected/focused, opens its third
+  argument with `tabopen -b` and so on and so forth until all arguments have
+  been opened in a new tab or until a tab is closed without being
+  activated/selected/focused.
+
+  Example usage:
+    `tabqueue http://example.org http://example.com http://example.net`
+    `composite hint -qpipe a href | tabqueue`
+*/
+//#background
+export function tabqueue(...addresses: string[]) {
+    // addresses[0] is a string when called with `tabopen a b c` but an array
+    // when called from `composite hint -qpipe a href | tabqueue`.
+    addresses = addresses.flat(Infinity);
+    if (addresses.length === 0) {
+        return Promise.resolve();
+    }
+    return tabopen("-b", addresses[0]).then(tab =>
+        new Promise ((resolve, reject) => {
+            function openNextTab(activeInfo) {
+                if (activeInfo.tabId === tab.id) {
+                    resolve(tabqueue(...(addresses.slice(1))));
+                    removeTabqueueListeners(tab.id);
+                }
+            }
+            function removeTabqueueListeners(tabId) {
+                if (tabId === tab.id) {
+                    browser.tabs.onActivated.removeListener(openNextTab);
+                    browser.tabs.onRemoved.removeListener(removeTabqueueListeners);
+                    // FIXME: This should actually be `reject(tab)` to
+                    // interrupt pipelines, but this results in an impossible
+                    // to debug `Error: undefined` message being printed on the
+                    // command line. So we silently resolve the promise and
+                    // hope for the best.
+                    resolve(tab);
+                }
+            }
+            browser.tabs.onActivated.addListener(openNextTab);
+            browser.tabs.onRemoved.addListener(removeTabqueueListeners);
+        })
+    );
+}
+
 /** Resolve a tab index to the tab id of the corresponding tab in this window.
 
     @param index

src/lib/webext.ts

@@ -190,11 +190,12 @@ export async function openInTab(tab, opts = {}, strarr: string[]) {
     let engine = searchEngines.find(engine => engine.alias === firstWord)
     // Maybe firstWord is the name of a firefox search engine?
     if (engine !== undefined) {
-        return browserBg.search.search({
+        browserBg.search.search({
             tabId: tab.id,
             engine: engine.name,
             query: rest,
         })
+        return tab
     }
 
     // Maybe it's a domain without protocol
@@ -234,14 +235,16 @@ export async function openInTab(tab, opts = {}, strarr: string[]) {
 
         engine = searchEngines.find(engine => engine.alias === enginename)
         if (engine !== undefined) {
-            return browserBg.search.search({
+            browserBg.search.search({
                 tabId: tab.id,
                 engine: engine.name,
                 query: queryString,
             })
+            return tab
         }
     }
 
     // No search engine has been defined in Tridactyl, let's use firefox's default search engine
-    return browserBg.search.search({ tabId: tab.id, query: queryString })
+    browserBg.search.search({ tabId: tab.id, query: queryString })
+    return tab;
 }