Bugfix FXIOS-10584 - [Toolbar Redesign] 20+ new homepage tabs are opened when I try to open just one new tab (backport #23253)

:scroll: Tickets

After investigating, I've found out that the bug was caused by this PR that fixed FXIOS-10500. I'm not really 100% sure what was happening but we are dealing with references here and some kind of retain cycle was certainly occuring because after tapping the + (add new tab button) intermittently, 3, 4(a random number) of redux actions for opening a new tab were triggered.
While trying to find the cause of this bug, I found an even bigger memory leak for Toolbar Buttons that were never deallocated because self was captured in the action handler.
Reverting the change from the PR mentioned above and fixing the leak seemed to solve both problems. FXIOS-10500 and the one addressed in this PR.

You have to check all boxes before merging

[x] Filled in the above information (tickets numbers and description of your work)
[x] Updated the PR name to follow our PR naming guidelines
[ ] Wrote unit tests and/or ensured the tests suite is passing
[ ] When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
[ ] If needed, I updated documentation / comments for complex code and public methods
[ ] If needed, added a backport comment (example @Mergifyio backport release/v120)

This is an automatic backport of pull request #23253 done by Mergify.

File	Coverage
ToolbarButton.swift	0.0%	⚠️
BrowserAddressToolbar.swift	0.0%	⚠️