Bug 888773 - Sidebar followup: show/hide should take a window argument

Nit: in that case you can use `then(done, assert.fail)`; the `catch` is useful when you have to add a `then(null, assert.fail)`.