ES6 promises like using undefined as their first argument instead of null. Both should behave the same behavior

We compile using the alwaysStrict flag so these directives are not needed

- avoid public modifier
- use Disposable where applicable
- fix some event handler leaks
- clean up some TODO@ben