Any reason we shouldn't just define `parseAjaxResponse` here instead of up above? Also, why is `connection` a parameter? Doesn't `connection === this` in every place it's called?