In SERVER-90992 we had to leave the constructor for const Collection * untouched to avoid a lot of code churn.

Originally the intent was to replace it with a static method with the word UNSAFE written somewhere in its name to signal old users that they should replace its usages with the safer constructor.

This ticket is about considering whether to go through the churn and open TODO tickets in the leftover places or not.