We were not actually doing anything on the client thread, since
the vnc client had been created as a child of the server, and
thus could not be moved. Fixing this revealed a bug in the
cleanup logic. We need to make sure we deleteLater() the
client objects and we need to quit and delete the client
thread from the main thread, after the client has been
deleted.