e.g, if I enable "Break on all exceptions" and run this program, web inspector hangs for me. My guess is it's trying to do stuff above the JS stack at the time of stack overflow, which if true, is definitely not allowed. ``` let arg = 20 function f() { f(arg); } function recurse() { try { f(); } catch(e) { } setTimeout(recurse, 0); } recurse(); ```
<rdar://problem/29870873>
Created attachment 313459 [details] [PATCH] Proposed Fix v4
Comment on attachment 313459 [details] [PATCH] Proposed Fix v4 r=me Can you add a test that ensures we don't catch it in inspector?
Comment on attachment 313459 [details] [PATCH] Proposed Fix v4 Oops, yes I totally meant to add a test.
Created attachment 313546 [details] [PATCH] Proposed Fix - With Tests
Comment on attachment 313546 [details] [PATCH] Proposed Fix - With Tests View in context: https://bugs.webkit.org/attachment.cgi?id=313546&action=review > LayoutTests/inspector/debugger/no-pause-stack-overflow-exception.html:21 > + WebInspector.debuggerManager.assertionsBreakpoint.disabled = false; I can drop the assertions breakpoint. I really just need the all exceptions one.
Created attachment 313547 [details] [PATCH] Proposed Fix - With Tests
Comment on attachment 313547 [details] [PATCH] Proposed Fix - With Tests Clearing flags on attachment: 313547 Committed r218652: <http://trac.webkit.org/changeset/218652>
All reviewed patches have been landed. Closing bug.