Add SetAbortOnUncaughtExceptionCallback API

test/cctest/test-abort-on-uncaught-exception.cc

+// Copyright 2013 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "src/v8.h"
+
+#include "test/cctest/cctest.h"
+
+const char* BOOM_FUNC = "function boom() { throw new Error(\"boom\") }";
+
+TEST(AbortOnUncaughtExceptionDefault) {
+  v8::Isolate* isolate = CcTest::isolate();
+  v8::HandleScope handle_scope(isolate);
+  v8::Handle<v8::ObjectTemplate> global_template =
+      v8::ObjectTemplate::New(isolate);
+  LocalContext env(NULL, global_template);
+
+  i::FLAG_abort_on_uncaught_exception = true;
+  CompileRun(BOOM_FUNC);
+
+  v8::Local<v8::Object> global_object = env->Global();
+  v8::Local<v8::Function> foo =
+      v8::Local<v8::Function>::Cast(global_object->Get(v8_str("boom")));
+
+  USE(foo);
+  /*
+  Ideally, I'd like to use EXPECT_EXIT like following:
+
+  EXPECT_EXIT(foo->Call(global_object, 0, NULL);,

[Michael Starzinger, 2015/10/05 16:13:57]

There isn't really a way I can think of to model this with our cctests. The only
way I can think of would be to add a permanent test expectation, but I don't
like that approach. So I would be fine with dropping this test completely.

[julien.gilli, 2015/10/05 17:24:25]

Done.

+    ::testing::KilledBySignal(SIGKILL),
+    "throwing and using --abort-on-uncaught-exception should cause process "
+    "to abort");
+  */
+}
+
+
+bool NoAbortOnUncaughtException(v8::Isolate* isolate) { return false; }

[Michael Starzinger, 2015/10/05 16:18:39]

Also, we could check that this callback was actually called once, by
incrementing a static count initialized to 0 and then checking at the end of the
test case that {count == 1} holds.

[julien.gilli, 2015/10/05 17:24:25]

Done.

+
+
+TEST(AbortOnUncaughtExceptionUncaughtExceptionCallbackNoAbort) {

[Michael Starzinger, 2015/10/05 16:13:57]

If we are left with only one test case, then I suggest moving it to test-api.cc
instead, adding a new cc-file doesn't seem to be worth it in that case.

[julien.gilli, 2015/10/05 17:24:25]

Done.

+  v8::Isolate* isolate = CcTest::isolate();
+  v8::HandleScope handle_scope(isolate);
+  v8::Handle<v8::ObjectTemplate> global_template =
+      v8::ObjectTemplate::New(isolate);
+  LocalContext env(NULL, global_template);
+
+  i::FLAG_abort_on_uncaught_exception = true;
+  isolate->SetAbortOnUncaughtExceptionCallback(NoAbortOnUncaughtException);
+
+  CompileRun(BOOM_FUNC);
+
+  v8::Local<v8::Object> global_object = env->Global();
+  v8::Local<v8::Function> foo =
+      v8::Local<v8::Function>::Cast(global_object->Get(v8_str("boom")));
+
+  foo->Call(global_object, 0, NULL);
+}