dummy: Tidy up code in PersonaStore.remove_persona()

It was doing twice as many HashMap operations as strictly necessary,
and was using a potentially incorrect pointer in the signal emission
(i.e. using the Persona pointer provided by the caller, rather than
the one which was just removed from the personas map).