Move NJClient::Disconnect() m_downloads.Empty()

Clearing m_downloads requires first deleting each RemoteDownload and
then emptying the list itself.  Make these two steps adjacent in the
source code to eliminate the risk of accessing a freed RemoteDownload
before m_downloads.Empty() is called.

Signed-off-by: Stefan Hajnoczi <stefanha@gmail.com>