refactor: remove unused end param in ethereum-chain-util helpers

[jiexi]

Description

Removes unused end param in the ethereum-chain-util helpers

• validateChainId
• validateAddEthereumChainParams
• validateSwitchEthereumChainParams

Extending E2E timeout to get past "no timings found" error:

flags = { "circleci": { "timeoutMinutes": 30 } }

Related issues

See: #27847 (comment)

Manual testing steps

1. Go to this page...

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.

[adonesky1]

LGTM

[Gudahtt]

LGTM!

[metamaskbot]

Builds ready [1c7ec8c]

• 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
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
  • content-script: 0
  • offscreen: 0

Page Load Metrics (1720 ± 97 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	230	2400	1656	380	183
		domContentLoaded	1452	2350	1692	194	93
		load	1470	2405	1720	202	97
		domInteractive	24	138	43	28	14
		backgroundConnect	7	78	33	19	9
		firstReactRender	16	91	36	24	12
		getState	6	56	14	13	6
		initialActions	0	44	2	10	5
		loadScripts	1056	1832	1256	170	81
		setupStore	6	72	14	17	8
		uiStartup	1663	2733	1965	238	114

[Gudahtt]

This was hitting E2E test timeouts due to lack of timing data, causing too many long e2e test suites to be allocated to a single runner.

I've added a flag to the PR description to extend the E2E test timeout, and it is passing now.

[metamaskbot]

Builds ready [313291f]

• 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
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13
  • content-script: 0
  • offscreen: 0

Page Load Metrics (1620 ± 56 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1454	1843	1618	112	54
		domContentLoaded	1412	1789	1588	111	53
		load	1452	1851	1620	116	56
		domInteractive	20	128	44	27	13
		backgroundConnect	11	59	32	17	8
		firstReactRender	15	63	29	19	9
		getState	5	64	15	16	8
		initialActions	0	1	0	0	0
		loadScripts	1042	1367	1188	98	47
		setupStore	6	10	8	1	1
		uiStartup	1647	2058	1819	123	59

Bundle size diffs [🚀 Bundle size reduced!]

• background: -8 Bytes (-0.00%)
• ui: 0 Bytes (0.00%)
• common: 0 Bytes (0.00%)