138363 2014-11-04 11:21:27 -0800 Web Inspector: When pausing on exception from console we should show the source and call frames 2016-05-02 21:46:31 -0700 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 joepeck webkit-unassigned ggaren graouts joepeck timothy webkit-bug-importer oldest_to_newest 1046117 0 joepeck 2014-11-04 11:21:27 -0800 * SUMMARY Should show source code and call frames for exceptions triggered by console/eval evaluations. * STEPS TO REPRODUCE 1. Inspect any page. 2. Enable Pause on all exceptions 3. js> setTimeout(function(){[].x.x}) 4. When paused, hover over any JavaScript, e.g. "document" in the index.html => Exception, no Call Frames, no Source with the exception * NOTES Normally we disallow showing __WebInspectorConsole__ sources, but I think we should show it here. 1046118 1 webkit-bug-importer 2014-11-04 11:21:54 -0800 <rdar://problem/18869487> 1046119 2 joepeck 2014-11-04 11:24:18 -0800 Maybe we should do this if there is a __WebInspectorConsole__ in the call frames, but not if there are __WebInspectorInjectedScript__, which should be internal only. In fact, I think we had a workaround in the frontend to auto-continue in those cases, which we should add back. See related: <https://webkit.org/b/138341> Web Inspector: Inspector pauses 3 times when evaluating window.decodeURI('%%%') 1046180 3 timothy 2014-11-04 14:31:28 -0800 The idea of not pausing for exceptions from the console is to make the REPL be very friendly to errors. You should see the exception Error logged as the result. For timeouts, that is tricky and likely an edge case. I don't want to make the common case of getting an immediate exception cause the debugger to pause in a __WebInspectorConsole__ source resource. 1189619 4 joepeck 2016-05-02 18:51:06 -0700 Tim, I think you effectively fixed this by showing the Console Evaluation script. 1189674 5 timothy 2016-05-02 21:46:31 -0700 Yep.