Pinning tabs with multiple windows open results in duplicates

[petemill]

Description

Pinning tabs often causes those tabs to be pinned multiple times, especially if there is more than 1 window open. This is likely because updatePinnedTabs fires cyclically, i.e:
1. A tab is pinned in windowA
2. updatePinnedTabs is called to ensure all other windows get the pinned tabs
3. windowB has the tab pinned, which causes…
4. updatePinnedTabs to be called
5. State is not necessarily up to date, so windowB or windowC gets the tab pinned again

Steps to Reproduce

1. Open window1
2. Open window2
3. Open a tab in window1
4. Pin a tab in window1

This can then cause some weird errors when a window's tab state is out of sync with what it should have.

Actual result:

• The tab is pinned twice in window2 and once in window1

Expected result:

• The tab is pinned once in window2 and once in window1

Reproduces how often:

Brave Version

about:brave info:
master (0.22)

Reproducible on current live release:
No