Yeah, I think this is better. I think it could go either way, but eventually the InstallExtensionDialog isn't going to be directly responsible for this notification anyway, and it will definitely come through the model (via ExtensionManager) at that point.