chore: Optimize performance of Assets hooks

[darkwing]

Memoizes much of the work in the useAccountTotalCrossChainFiatBalance and useAccountTotalFiatBalance hooks

Description

Related issues

Fixes:

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.

[darkwing]

Do we even need to do this? Is the todo relevant?

[Prithpal-Sooriya]

If we can confirm that this has a ticket, I'm down to remove this TODO comment.

[Prithpal-Sooriya]

Nice work memoizing all of these!

When I did some investigating, I saw that this function uses an internal class Numeric. This has very high GC rate (since it keeps creating new objects).
^ Just noting this, we don't need to tackle GCs yet.

[metamaskbot]

Builds ready [816e74e]

• 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 (2022 ± 141 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1630	2637	2010	281	135
		domContentLoaded	1576	2456	1951	250	120
		load	1628	2662	2022	294	141
		domInteractive	25	111	45	25	12
		backgroundConnect	17	334	77	78	37
		firstReactRender	15	113	35	28	13
		getState	6	138	37	31	15
		initialActions	0	1	0	0	0
		loadScripts	1187	1867	1483	196	94
		setupStore	9	73	22	21	10
		uiStartup	1872	3882	2401	524	251

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 482 Bytes (0.01%)
• common: 0 Bytes (0.00%)