PSPromotionManager has a bunch of member functions and one data member that all have names with a _depth suffix. It's unclear what that's about. A guess is that it has something to do with depth-first processing, but if so then it seems pointless to have such a naming convention if there isn’t a related breadth-first variant. Maybe there was such a thing a long time ago? In the absence of such, we should remove the suffix.