JS errors thrown in private scripts should be reported to stderr in Debug builds

Source/bindings/templates/methods.cpp

Index: Source/bindings/templates/methods.cpp
diff --git a/Source/bindings/templates/methods.cpp b/Source/bindings/templates/methods.cpp
index a7733bc9d8386b7c2f60793b417a4f9d0b38f431..0413a1cfb88410af8048eb78e7e8d4fa4b39ce03 100644
--- a/Source/bindings/templates/methods.cpp
+++ b/Source/bindings/templates/methods.cpp
@@ -525,6 +525,9 @@ bool {{v8_class}}::PrivateScript::{{method.name}}Method({{method.argument_declar
     {% endif %}
     ExceptionState exceptionState(ExceptionState::ExecutionContext, "{{method.name}}", "{{cpp_class}}", scriptState->context()->Global(), scriptState->isolate());
     v8::TryCatch block;
+#ifndef NDEBUG
+    block.SetVerbose(true);
+#endif
     V8RethrowTryCatchScope rethrow(block);

[Jens Widell, 2014/08/11 09:26:15]

Unrelated: Using a V8RethrowTryCatchScope here is a bit weird, actually, since
any exception initially caught by this TryCatch must explicitly *not* be
rethrown; we either throw another exception, or die. Having it here doesn't
cause any wrong behavior, but perhaps it would be clearer in this particular
case to have an explicit block.ReThrow() once we've thrown the correct
exception?

[haraken, 2014/08/11 09:41:16]

Makes a lot of sense. Will fix in a separate CL shortly.

     {% if method.idl_type == 'void' %}
     PrivateScriptRunner::runDOMMethod(scriptState, "{{cpp_class}}", "{{method.name}}", holder, {{method.arguments | length}}, argv);