[charts] Use Map for string cache instead of object

[bernardobelchior]

Use Map for string cache instead of object. We get entry counting for free.

Also removed unstable_cleanupDOM since it wasn't doing anything.

[github-actions]

Thanks for adding a type label to the PR! 👍

[mui-bot]

Deploy preview: https://deploy-preview-17982--material-ui-x.netlify.app/

Bundle size report

Total Size Change:^{${\tiny{\color{green}▼}}$}-1.58KB^(-0.01%) - Total Gzip Change:^{${\tiny{\color{green}▼}}$}-431B^(-0.01%)
Files: 118 total (0 added, 0 removed, 17 changed)

Show 17 more bundle changes

@mui/x-charts-pro/Heatmap parsed:^{${\tiny{\color{green}▼}}$}-98B^(-0.05%) gzip:^{${\tiny{\color{green}▼}}$}-20B^(-0.03%)
@mui/x-charts/ChartsAxis parsed:^{${\tiny{\color{green}▼}}$}-95B^(-0.13%) gzip:^{${\tiny{\color{green}▼}}$}-19B^(-0.08%)
@mui/x-charts/ChartsXAxis parsed:^{${\tiny{\color{green}▼}}$}-95B^(-0.14%) gzip:^{${\tiny{\color{green}▼}}$}-14B^(-0.06%)
@mui/x-charts/ChartsYAxis parsed:^{${\tiny{\color{green}▼}}$}-95B^(-0.14%) gzip:^{${\tiny{\color{green}▼}}$}-15B^(-0.06%)
@mui/x-charts parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.04%) gzip:^{${\tiny{\color{green}▼}}$}-33B^(-0.04%)
@mui/x-charts-pro parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.03%) gzip:^{${\tiny{\color{green}▼}}$}-27B^(-0.03%)
@mui/x-charts-pro/BarChartPro parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.04%) gzip:^{${\tiny{\color{green}▼}}$}-26B^(-0.04%)
@mui/x-charts-pro/FunnelChart parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.05%) gzip:^{${\tiny{\color{green}▼}}$}-30B^(-0.05%)
@mui/x-charts-pro/LineChartPro parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.04%) gzip:^{${\tiny{\color{green}▼}}$}-20B^(-0.03%)
@mui/x-charts-pro/ScatterChartPro parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.05%) gzip:^{${\tiny{\color{green}▼}}$}-39B^(-0.06%)
@mui/x-charts/BarChart parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.05%) gzip:^{${\tiny{\color{green}▼}}$}-25B^(-0.04%)
@mui/x-charts/ChartsReferenceLine parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.15%) gzip:^{${\tiny{\color{green}▼}}$}-26B^(-0.12%)
@mui/x-charts/Gauge parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.09%) gzip:^{${\tiny{\color{green}▼}}$}-18B^(-0.05%)
@mui/x-charts/LineChart parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.05%) gzip:^{${\tiny{\color{green}▼}}$}-39B^(-0.06%)
@mui/x-charts/RadarChart parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.06%) gzip:^{${\tiny{\color{green}▼}}$}-25B^(-0.05%)
@mui/x-charts/ScatterChart parsed:^{${\tiny{\color{green}▼}}$}-92B^(-0.06%) gzip:^{${\tiny{\color{green}▼}}$}-36B^(-0.07%)
@mui/x-charts/ChartsText parsed:^{${\tiny{\color{green}▼}}$}-89B^(-2.93%) gzip:^{${\tiny{\color{green}▼}}$}-19B^(-1.26%)

Details of bundle changes

Generated by 🚫 dangerJS against 88a0540

[codspeed-hq]

CodSpeed Performance Report

Merging #17982 will not alter performance

_{Comparing bernardobelchior:use-map-string-cache (88a0540) with master (2583a52)}

Summary

✅ 9 untouched benchmarks