javascriptですべてのタブをブックマークして閉じるにはどうすればいいですか？

Question

現在のタブのブックマークを保存し、WebExtensionボタンを押したときに閉じてしまうコードがあります。コードを保存してからすべてのタブを閉じるようにします。

var currentTab; 
var currentBookmark; 

// gets active tabe 
function callOnActiveTab(callback) { 
    chrome.tabs.query({currentWindow: true}, function(tabs) { 
     for (var tab of tabs) { 
     if (tab.active) { 
      callback(tab, tabs); 
     } 
     } 
    }); 
} 

/* 
* Add the bookmark on the current page. 
*/ 
function Bookmark() { 

    chrome.bookmarks.create({title: currentTab.title, url: currentTab.url}, function(bookmark) { 
     currentBookmark = bookmark; 
    }); 

    callOnActiveTab((tab) => { 
     chrome.tabs.remove(tab.id); 
    }); 

} 

/* 
* Switches currentTab and currentBookmark to reflect the currently active tab 
*/ 
function updateTab() { 
    chrome.tabs.query({active: true, currentWindow: true}, function(tabs) { 
    if (tabs[0]) { 
     currentTab = tabs[0]; 

     chrome.bookmarks.search({url: currentTab.url}, (bookmarks) => { 
     currentBookmark = bookmarks[0]; 
     }); 
    } 
    }); 
} 



function listTabs() { 

    Bookmark(); 

} 


chrome.browserAction.onClicked.addListener(listTabs); 


chrome.tabs.onUpdated.addListener(updateTab); 
// listen to tab switching 
chrome.tabs.onActivated.addListener(updateTab); 

私はupdateTab（）関数の最後にしおり（）関数を追加した場合、ボタンはもはや機能していないと私はタブを変更する場合には、1とすると、すべてのタブを終了することを保存します。

Answer 1

あなたのコードは、あなたがしようとしているように思われるほど複雑すぎます。 Bookmark関数を使用して複数のタブをブックマークして削除することができないという問題の重要な部分は、アクティブなタブをトラッキングしている非同期イベントハンドラによって変更されるグローバル変数に依存していることです。その関数は、関数に渡される引数を使用するように再コーディングできます。このようにして、一般的に再利用することができます。

注：bookmarkTab関数のタブの削除を移動しました（あなたのコードではBookmarkとなります）。そこにあるのは、関数Bookmarkだけを呼び出すのは悪い考えです。私はbookmarkAndRemoveTab()関数を追加しました。これは、実行中の両方の作業に対して明示的に指定されています。

はちょうどあなたの browserActionに関連したセクションは次のようになります。

/* Bookmark and remove all tabs 
*/ 
function bookmarkAndRemoveAllTabs() { 
    //Get all tabs in 'normal' windows: 
    // May want to test. Could want to get all tabs in all windows 
    // Of windowTypes:["normal","popup","panel","devtools", probably only 
    // want "normal" and "popup" tabs to be bookmarked and closed. 
    let queryInfos = [{windowType: 'normal'},{windowType: 'popup'}]; 
    queryInfos.forEach(function(queryInfo){ 
     chrome.tabs.query(queryInfo, function(tabs) { 
      for (let tab of tabs) { 
       bookmarkAndRemoveTab(tab); 
      } 
     }); 
    }); 
}