shell: Avoid destroying modal windows in the "response" signal handler

Other code may legitimately connect to the "response" signal handler as
well and if we destroy it in the handler, their callbacks are never
invoked. Instead, do our window tracking in the "unmap" signal handler.

https://bugzilla.gnome.org/show_bug.cgi?id=764437