fix: Fix EditAccountsModal and EditNetworkModal checkboxes reseting back to default on rerender

[jiexi]

Description

Fixes a bug in the EditAccountsModal and EditNetworkModal that occasionally causes any changes to the default selected checkboxes to be reset on component rerender.

Related issues

See: #27847 (comment)

Manual testing steps

1. For a dapp with permissions
2. In the wallet UI, edit the dapp permissions
3. Open the edit network modal
4. Change some checkboxes to be different from what was default selected
5. Wait
6. The checkboxes should not revert back to the default selected

Do the same for the edit accounts modal

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.

[metamaskbot]

Builds ready [87517f8]

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

Page Load Metrics (1668 ± 83 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	330	2010	1530	400	192
		domContentLoaded	1353	1962	1645	170	82
		load	1371	2013	1668	173	83
		domInteractive	23	76	43	19	9
		backgroundConnect	7	66	22	18	9
		firstReactRender	16	74	36	21	10
		getState	4	75	21	22	11
		initialActions	0	1	0	0	0
		loadScripts	966	1472	1211	142	68
		setupStore	6	47	11	11	5
		uiStartup	1554	2416	1912	252	121

Bundle size diffs

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

[Gudahtt]

LGTM!