DomInspector, ToolboxOverlay.attach calls target.client.listTabs twice

[groovecoder]

Some of these issues are just me learning how this code works, so feel free to close them without "fixing" ...
toolbox-overlay.js calls listTabs:

  target.client.listTabs(response => {

Then later calls it again inside the arrow function:

    target.client.listTabs(({ tabs, selected }) => {

If I understand correctly, the 2nd call could be:

    target.client.listTabs((response.tabs, response.selected) => {

❓

That would eliminate a duplicate call to listTabs right?

[groovecoder]

We could add this to the top of the arrow function to keep function argument names short:

let { tabs, selected } = response;