When coordinator leaves (CoordGmsImpl#leave) it puts LEAVE request in the queue, and then it calls stop(true) on view handler. This in turn flips boolean closed of the queue to true - the same boolean that is checked in ViewHandler#run while statement.

If ViewHandler happens to be processing requests in ViewHandler#process(List requests) when LEAVE gets enqueued then ViewHandler promptly exists (due to boolean q.isClosed() in while statement) without actually processing pending LEAVE or other requests!

We should use while(true) in ViewHandler#run and have ViewHandler die gracefully once it pops out poison pill (end marker) and throws QueueClosedException.