Nit: this indentation seems a little weird to me--maybe JSLint is forcing you to do this? (I'd probably use a workaround like aligning the "done" with this, which unfortunately leads to double-indentation.)