ArcRotateCamera: Modified zoomOn to use same logic as FramingBehavior

[PolygonalSun]

A user on the forum found that the zoomOn function was found to be incorrectly determining radius for zooming on provided meshes. This PR contains a change that takes some of the logic used for the ArcRotateCamera framing behavior and uses that to determine how far to zoom.

Forum Link: https://forum.babylonjs.com/t/arcrotate-camera-zoomon-not-working-as-expected/38130/1

[bjsplat]

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

[bjsplat]

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

[bjsplat]

Snapshot stored with reference name:
refs/pull/14387/merge

Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14387/merge/index.html

To test a playground add it to the URL, for example:

https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14387/merge/index.html#WGZLGJ#4600

Links to test babylon tools with this snapshot:

https://playground.babylonjs.com/?snapshot=refs/pull/14387/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14387/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14387/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14387/merge

To test the snapshot in the playground with a playground ID add it after the snapshot query string:

https://playground.babylonjs.com/?snapshot=refs/pull/14387/merge#BCU1XR#0

[sebavan]

If the frustrumSlope function is only use in this context, I would suggest to call it from within LowerRadius... instead of separately outside ?

[RaananW]

Just two comments, otherwise - looks good