I prefer an early exit to wrapping the whole function body in an `if`. It looks more like a guard that way. Also, I prefer to use `XMLHttpRequest::DONE`.

```coffee
return unless xhr.readyState is xhr.DONE
```