Extension API: chrome.tabs.remove

[jonathansampson]

https://developer.chrome.com/extensions/tabs#method-remove

Blocks #8327, #9330, #9531, #9654, #9828

[bridiver]

closed by brave/muon@cca3034

[jonathansampson]

This appears to have regressed; I'm unable to successfully close a tab in the following build:

Brave: 0.19.0
rev: 2bd7726
Muon: 4.2.6
libchromiumcontent: 59.0.3071.109
V8: 5.9.211.38
Node.js: 7.9.0
Update Channel: dev
os.platform: win32
os.release: 10.0.15063
os.arch: x64

Need to support the following patterns:

• chrome.tabs.remove(23)
• chrome.tabs.remove([23, …])
• chrome.tabs.remove(23, function () {})
• chrome.tabs.remove([23, …], function () {})

[jonathansampson]

This issue appears to impact LastPass 2FA tabs: #9259

[bridiver]

I think we stopped handling close in browser-laptop because nothing has changed in muon. The original change to fix the crash no longer makes sense with the changes we made in browser-laptop so I'll commit a workaround to browser-laptop and then update muon