fix: Prevent SrpViewSrpText from firing more than once

[darkwing]

Description

Ensures that the SrpViewSrpText event doesn't get reported more than once despite rerouting triggered by other events.

Related issues

Fixes:

Manual testing steps

1. Go to Settings -> Security -> Reveal Seed Phrase
2. Complete quiz and enter your password
3. Ensure that the event is only tracked once.

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[darkwing]

This is a temporary measure; we need to figure out why the app is rendering this so often. I think it's due to the location re-renders I pointed out a few weeks ago:

[metamaskbot]

Builds ready [0c4db27]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • content-script: 0
  • offscreen: 0

Page Load Metrics (1773 ± 63 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	354	2082	1700	335	161
		domContentLoaded	1521	2004	1745	126	61
		load	1529	2075	1773	131	63
		domInteractive	27	80	39	15	7
		backgroundConnect	10	69	35	20	9
		firstReactRender	15	73	32	21	10
		getState	5	51	19	15	7
		initialActions	0	1	0	0	0
		loadScripts	1112	1578	1303	114	55
		setupStore	8	66	16	15	7
		uiStartup	1742	2304	2020	143	69

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 119 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[EtherWizard33]

LGTM

[EtherWizard33]

expected console.log, correct?

[gambinish]

Doesn't seem like this log is needed? Can it be removed?

[darkwing]

Drp, sorry. Will do.

[metamaskbot]

Builds ready [8e23bc3]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • content-script: 0
  • offscreen: 0

Page Load Metrics (1837 ± 55 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	426	2011	1763	324	156
		domContentLoaded	1598	1998	1807	109	53
		load	1608	2013	1837	115	55
		domInteractive	16	82	38	16	7
		backgroundConnect	12	65	35	22	10
		firstReactRender	16	74	38	23	11
		getState	5	67	17	17	8
		initialActions	0	1	0	0	0
		loadScripts	1171	1523	1347	102	49
		setupStore	9	63	25	20	10
		uiStartup	1846	2312	2105	133	64

[metamaskbot]

Builds ready [7b69c29]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • content-script: 0
  • offscreen: 0

Page Load Metrics (2054 ± 107 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	497	2698	1968	400	192
		domContentLoaded	1692	2616	2005	203	98
		load	1740	2712	2054	222	107
		domInteractive	29	86	44	15	7
		backgroundConnect	12	160	55	41	20
		firstReactRender	15	70	35	21	10
		getState	6	61	23	18	9
		initialActions	0	1	0	0	0
		loadScripts	1274	2022	1519	172	83
		setupStore	9	57	16	14	7
		uiStartup	1928	3174	2328	312	150

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 65 Bytes (0.00%)
• common: 0 Bytes (0.00%)