Handling of infinite collection initialization chains could be unified

[jjonescz]

ERR_CollectionInitializerInfiniteChainOfAddCalls and ERR_ParamsCollectionInfiniteChainOfConstructorCalls

See #74899 (comment):

We are handling cycles through constructors (ERR_ParamsCollectionInfiniteChainOfConstructorCalls) elsewhere and with a different approach. I assume that at the time that handling was introduced, there was no way to get into a cycle through an Add method. It feels like we should unify and combine both checks, i.e. do them in the same place and use similar approaches for both.