fix: Network filter must respect PORTFOLIO_VIEW feature flag

[gambinish]

Description

When feature flag is omitted, we should not show tokens across all chains, only the currentNetwork.chainId

Related issues

Fixes: Multichain tokens being rendered when PORTFOLIO_VIEW feature is off (unexpected)

Manual testing steps

With feature flag off:

yarn webpack --watch

When running the app, the tokens displayed should always belong to the globally selected network.

With feature flag on:

PORTFOLIO_VIEW=1 yarn webpack --watch

When running the app, you should see all tokens from all added networks, when "All Networks" filter is selected. When "Current filter" is selected, you should see only tokens for the globally selected network.

Screenshots/Recordings

Before

After

Screen.Recording.2024-11-21.at.12.31.04.PM.mov

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 [109611f]

• 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 (1994 ± 78 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1767	2403	1992	164	79
		domContentLoaded	1754	2346	1960	148	71
		load	1768	2404	1994	163	78
		domInteractive	15	89	39	16	8
		backgroundConnect	11	159	31	33	16
		firstReactRender	96	384	150	72	34
		getState	5	63	23	21	10
		initialActions	0	1	0	0	0
		loadScripts	1266	1718	1446	122	59
		setupStore	6	75	18	21	10
		uiStartup	1982	2777	2293	225	108

Bundle size diffs

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