GeometryVisualizer subscribe once per entity, not once per geometry

[hpinkos]

Leftover from #6239

Instead of listening to entity.definitionChanged once per geometry type, this listens to entity.definitionChanged once and loops through each geometry type when it is trigggered.

[cesium-concierge]

@hpinkos, thanks for the pull request! Maintainers, we have a signed CLA from @hpinkos, so you can review this at any time.

⚠️ I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

---

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

[mramato]

Looks like you have some failing tests.

[hpinkos]

@mramato ready

[mramato]

Thanks, not sure how much of a perceivable impact it will have, but this will significantly reduce the number of entity subscriptions by 10x, and that should lead to less memory usage and way less iteration when something changes..