[All] Clean collision detection in FreeMotionAnimationLoop

[bakpaul]

The inheritance link between CollisionResetVisitor and CollisionVisitor and some other had a consequence that ProcessGeometricalData was called multiple times in the FreeMotionAnimationLoop when the free motion and the collision detection where computed in parallel.

Another issue was that we had two time the timers for the substeps of the collision detection in the case of parallel computation : IMO only the AnimationLoop should have timers for global computation made by visitors. Here I remove the one of the Pipeline.

---

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).

---

Reviewers will merge this pull-request only if

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[bakpaul]

[ci-build][with-all-tests]

[fredroy]

[ci-build][with-all-tests][force-full-build]

[alxbilger]

I don't see any implementation of processGeometricalData. Could you give more details on this function? The documentation is not very helpful

[bakpaul]

It was introduced by @courtecuisse to be able to perform some processing before the collision detection to speed up the process of proximity computation in his plugins of collision detection.