New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(ios): handle uncaught exceptions in timers and Obj-C based proxy fireEvent lifecycle #11439
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is probably an unnecessary complaint, but i really got to dislike all those damn Obj-C square brackets so i try to get rid of them whenever possible 😅
iphone/TitaniumKit/TitaniumKit/Sources/Kroll/KrollTimerManager.m
Outdated
Show resolved
Hide resolved
iphone/TitaniumKit/TitaniumKit/Sources/Kroll/KrollTimerManager.m
Outdated
Show resolved
Hide resolved
iphone/TitaniumKit/TitaniumKit/Sources/Kroll/KrollTimerManager.m
Outdated
Show resolved
Hide resolved
Fixes TIMOB-27528
Fixes TIMOB-27528
0f44091
to
bae7732
Compare
JIRA: https://jira.appcelerator.org/browse/TIMOB-27528
Description:
This adds some manual checking of unhandled exceptions after firing events or running timers using the new Obj-C JSC API. If there is an exception, we call out TiExceptionHandler code (i.e. display the grey/red stack trace view, fire Ti.App.uncaughtException event).
Note that last time we added an exception handler on the JSC API itself and that messed with the test suite working properly. I hope/assume this will not?