nit: our [style guide](https://github.com/mozilla/addon-sdk/wiki/Coding-style-guide) suggests doing:

    }
    else {