Fix potential for null activity being passed to adapter causing crashes

This occurs if we've left the view before we finished loading it. Perhaps later we should instead cancel the async task if we move away from the view? This would be better.
Instead of doing all these checks just cancel the loading task if we navigate away
Cancel the login task instead of just dismissing the dialog
Add loading task cancellation to other fragments