fix: Fix race condition validating ERC20 transfer (blockaid)

[vinistevam]

Description

Fixed race condition when validating ERC20 transfer and enabled ERC20 transfer blockaid e2e.

Related issues

Fixes: #28434

Manual testing steps

1. Go to this the test dapp
2. Click and reject (fast ) all types of requests in the section: PPOM - Malicious Transactions and Signatures
3. Should not get stuck in the loader

Screenshots/Recordings

Before

all.PPOM.signatures.webm

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.

[metamaskbot]

Builds ready [47a40ed]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (1917 ± 74 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1736	2294	1919	148	71
		domContentLoaded	1698	2119	1872	116	56
		load	1737	2285	1917	154	74
		domInteractive	28	89	43	14	7
		backgroundConnect	8	257	44	56	27
		firstReactRender	51	321	131	77	37
		getState	4	127	26	35	17
		initialActions	0	0	0	0	0
		loadScripts	1254	1605	1376	99	47
		setupStore	5	82	23	25	12
		uiStartup	1929	3003	2217	255	123

Bundle size diffs [🚨 Warning! Bundle size has increased!]

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

[metamaskbot]

Builds ready [d8b5076]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (2118 ± 192 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	423	3580	2030	539	259
		domContentLoaded	1614	3524	2091	397	191
		load	1619	3536	2118	399	192
		domInteractive	15	63	39	11	5
		backgroundConnect	7	100	32	26	13
		firstReactRender	53	310	133	63	30
		getState	4	96	28	27	13
		initialActions	0	1	0	0	0
		loadScripts	1099	2955	1550	368	177
		setupStore	5	54	13	13	6
		uiStartup	1811	3812	2403	465	223

Bundle size diffs [🚨 Warning! Bundle size has increased!]

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