DevTools: autoresume execution on parse errors.

webkit/glue/devtools/js/debugger_agent.js

Index: webkit/glue/devtools/js/debugger_agent.js
===================================================================
--- webkit/glue/devtools/js/debugger_agent.js	(revision 27065)
+++ webkit/glue/devtools/js/debugger_agent.js	(working copy)
@@ -790,8 +790,9 @@
   WebInspector.currentPanel = WebInspector.panels.scripts;
 
   var body = msg.getBody();
-  if (this.pauseOnExceptions_) {
-    var body = msg.getBody();
+  // No script field in the body means that v8 failed to parse the script. We
+  // resume execution on parser errors automatically.
+  if (this.pauseOnExceptions_ && body.script) {

[Søren Thygesen Gjesse, 2009/09/25 06:28:46]

The assumption that if there is no script information then it is a parsing error
seems fragile. Changes to V8 might break that. I think we should aim for adding
information to the exception event which will make this check more robust.

[yurys, 2009/09/25 14:01:56]

Are there other cases when body.script is undefined except parsing errors or we
can just add something like body.isParsingError flag?

     var line = devtools.DebuggerAgent.v8ToWwebkitLineNumber_(body.sourceLine);
     this.createExceptionMessage_(body.script.name, line, body.exception.text);
     this.requestBacktrace_();