only set the activeFrameKey if the tab is already active

[bridiver]

Submitter Checklist:

• 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).
• Tagged reviewers and labelled the pull request as needed.

probable fix for #9134

Test Plan:
also fixes issue an mac where the first regular tab is restored instead of the pinned tab. Same QA steps as #9134 (minus the waiting)

Reviewer Checklist:

Tests

• Adequate test coverage exists to prevent regressions
• Tests should be independent and work correctly when run individually or as a suite ref
• New files have MPL2 license header

[bsclifton]

Change looks good- I added some unit tests to capture this new behavior 😄