History page now properly opens new tab w/ link that was clicked

[bsclifton]

• Submitted a ticket for my issue if one did not already exist.
• Used Github auto-closing keywords in the commit message.
• Added/updated tests for this change (for new code or code which already has tests).
• Ran git rebase -i to squash commits (if needed).

Fixes #4909

Auditors: @darkdh

Test Plan:

1. Launch Brave and visit a few sites
2. Visit about:history
3. Double click one of the entries and confirm it opens the location in a new tab

[bsclifton]

NOTE: I did not see any code change responsible for this. All that was changed was updating the code to use the immutable.js style accessors (versus directly accessing)

[darkdh]

I think it was 57086b8#diff-237402632b025505a0517d846e9e32c1R28

[darkdh]

++

[bbondy]

It wasn't because that commit is from 0.116 and this worked in 0.12.4. But probably someone changed the callback handling inside SortableTable or something similar within the last week.

[darkdh]

Sorry for the wrong reference. The malfunction started from this commit eea84e5

[bsclifton]

Good find @darkdh, that is indeed it. Unfortunately, this bug exists in 0.12.5 and was shipped ☹️

[bbondy]

we were aware just before shipping but decided not to cancel the release yesterday for a respin.