fix: Android crash when svgs use the " html entity

[joaoloureirop]

Description

Avoid crashes on malformed SVGs from URIs that use the HTML entity " instead of "
react-native-svg team acknowledges this parser needs more work to parse HTML entites correctly

Related issues

Fixes: #10614

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile 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.

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 54cabe0
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/a3ad5217-0c2a-491f-ad4b-707643efe9d4

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.75%. Comparing base (8dcdd3c) to head (6d60d36).
Report is 24 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10899      +/-   ##
==========================================
- Coverage   52.75%   52.75%   -0.01%     
==========================================
  Files        1534     1539       +5     
  Lines       36777    36852      +75     
  Branches     4335     4358      +23     
==========================================
+ Hits        19403    19442      +39     
- Misses      16058    16086      +28     
- Partials     1316     1324       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 730d860
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/77eb9bd3-54e1-46a2-8504-9c264f32ebf3

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

[Daniel-Cross]

Looks good!

[chrisleewilcox]

@joaoloureirop E2E is failing. Not seeing anything consistent. Probably just run it one more time to be sure.

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: d4df34c
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/29b825c4-68b3-46e5-bb4d-2e1378d930bb

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

[NicolasMassart]

Looks good to me.

[github-actions]

Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 190ccea
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/fde29bf4-6837-49ef-8dd6-57afaeaa0021

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud