Middle click on pinned tab causes other tabs to get pinned and loses the original pinned tab

[srirambv]

Test plan

1. Open a bunch of tabs and pin a few
2. Middle click on a pinned tab to close
3. Ensure other tabs are not affected by this

Description

Middle click on pinned tab causes other tabs to get pinned and looses the original pinned tab

Steps to Reproduce

1. Clean install 0.20.6
2. Open 4-5 tabs and visit sites in each of them
3. Pin one of the tabs
4. Middle click on the pinned tab, all other tabs get pinned momentarily and the original pinned tab is lost
   5. Relaunch browser, pinned tab is lost permanently

Actual result:

Expected result:
Should not do anything to pinned tabs or other open tabs

Reproduces how often:
100%

Brave Version

about:brave info:

Brave	0.20.6
V8	6.2.414.36
rev	18f7d1e
Muon	4.5.14
OS Release	10.0.15063
Update Channel	Beta
OS Architecture	x64
OS Platform	Microsoft Windows
Node.js	7.9.0
Brave Sync	v1.4.2
libchromiumcontent	62.0.3202.75

Reproducible on current live release:
N/A

Additional Information

[alexwykoff]

@srirambv could you give this another go? Thanks!

[petemill]

Just reproduced on master

[srirambv]

Side effect of this issue is captured here #12379

[cezaraugusto]

bug confirmed but btw closing the pinned tab with middle click is the expected behavior

[petemill]

surely if middle-click to close should work, then cmd(or ctrl)-w should also, since they do the same thing? It must be both or neither, I would imagine, since they are the same 'shortcut'.