Isn't OnDisconnect called within `SocketClosed()`, so it *is* called for dropped connections?

It should be removed because we already have the player destroying hook that is always fired.