I believe this is a relic from the past when we used not to destroy the ShellClient when the associated surface is unmapped.