Rename the rest of the internal events to the onDid* pattern

As I mentioned in another comment, my concern is that `Event Subscription` methods are also instance methods. Maybe we could squint and ignore that weirdness, but it seems weird. I'm less convinced than you are that event subscription methods need to be called out from other methods if there's not a ton of them.