Add stack trace to the promise reject callback.

src/isolate.cc

Index: src/isolate.cc
diff --git a/src/isolate.cc b/src/isolate.cc
index 7cf525d3e157a052dd78f21e25df1432518ee9c1..684f03e49cd9a37eb238d1920027d225054f890f 100644
--- a/src/isolate.cc
+++ b/src/isolate.cc
@@ -2272,8 +2272,17 @@ void Isolate::ReportPromiseReject(Handle<JSObject> promise,
                                   Handle<Object> value,
                                   v8::PromiseRejectEvent event) {
   if (promise_reject_callback_ == NULL) return;
-  promise_reject_callback_(v8::Utils::PromiseToLocal(promise),
-                           v8::Utils::ToLocal(value), event);
+  Handle<JSArray> stack_trace;
+  if (event == v8::kPromiseRejectWithNoHandler &&
+      capture_stack_trace_for_uncaught_exceptions_ && value->IsJSObject()) {

[aandrey, 2014/10/07 10:49:05]

I think we can drop the capture_stack_trace_for_uncaught_exceptions_ check:
1) will be still handled correctly below,
2) this flag may be false, but we could still have the stack collected.

+    Handle<JSObject> error_obj = Handle<JSObject>::cast(value);
+    Handle<Name> key = factory()->detailed_stack_trace_symbol();
+    Handle<Object> property = JSObject::GetDataProperty(error_obj, key);
+    if (property->IsJSArray()) stack_trace = Handle<JSArray>::cast(property);
+  }
+  promise_reject_callback_(v8::Utils::PromiseToLocal(promise), event,
+                           v8::Utils::ToLocal(value),
+                           v8::Utils::StackTraceToLocal(stack_trace));
 }