Extension APIs: chrome.browserAction.setIcon fails with numerical tabId

[jonathansampson]

When chrome.browserAction.setIcon is called with an explicit tabId, Brave fails to set the new image. The following steps will demonstrate this issue:

1. Enable Honey (it should start with a gray icon)
2. Navigate to Honey's background page
3. From the console, run Script A (seen below)
4. Note that the icon is still gray for all tabs
5. From the console, run Script B (seen below)

Script A:

chrome.tabs.query({}, tabs => {
    tabs.forEach(tab => {
        chrome.browserAction.setIcon({
            path: {
                19: "icons/active-19.png",
                38: "icons/active-38.png"
            },
            tabId: tab.id
        })
    })
})

Script B:

chrome.tabs.query({}, tabs => {
    tabs.forEach(tab => {
        chrome.browserAction.setIcon({
            path: {
                19: "icons/active-19.png",
                38: "icons/active-38.png"
            }
        })
    })
})

[jonathansampson]

Blocks #8118