fix(q.js): if rejection handler throws an exception, reject promise

[jamestalmage]

per Promise/A+ spec

[jamestalmage]

This (in conjunction with latest on master) will make the tests pass.

I'm suspicious that similar try/catch wrapping should be done in ref implementation here. But the tests pass without it.

[IgorMinar]

ref is currently never called or exposed in a way that would allow non-$q listeners to be registered with its promise. though it does make me a bit nervous having it there exposed waiting for someone to change $q.js in a way where the previous statement would not be true any more.

I'm going to close this as dupe of #3699 since I've already implemented this change and included tests.

thanks