[V8] Use Function.name for stack frames in v8::StackTrace

test/cctest/test-api.cc

Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index 907543ef548188395804bc57f22c7257ed657631..bbc5d4cffc43cf1cb833f4252836bdb1dee71521 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -14911,6 +14911,48 @@ TEST(CaptureStackTraceForUncaughtExceptionAndSetters) {
 }
 
 
+static void StackTraceDisplayNameListener(v8::Handle<v8::Message> message,
+                                          v8::Handle<Value>) {
+  v8::Handle<v8::StackTrace> stack_trace = message->GetStackTrace();
+  CHECK_EQ(4, stack_trace->GetFrameCount());
+  checkStackFrame("origin", "foo:0", 4, 7, false, false,
+                  stack_trace->GetFrame(0));
+  checkStackFrame("origin", "foo:1", 5, 27, false, false,
+                  stack_trace->GetFrame(1));
+  checkStackFrame("origin", "foo", 5, 27, false, false,
+                  stack_trace->GetFrame(2));
+  checkStackFrame("origin", "", 1, 14, false, false, stack_trace->GetFrame(3));
+}
+
+
+TEST(GetStackTraceContainsFunctionsWithDisplayName) {

[yurys, 2015/03/05 14:12:56]

...WithFunctionName

[kozy, 2015/03/05 15:01:09]

Done.

+  LocalContext env;
+  v8::HandleScope scope(env->GetIsolate());
+
+  CompileRunWithOrigin(
+      "function gen(name, counter) {\n"
+      "  var f = function foo() {\n"
+      "    if (counter === 0)\n"
+      "      throw 1;\n"
+      "    gen(name, counter - 1)();\n"
+      "  };\n"
+      "  Object.defineProperty(f, 'name', {writable:true});\n"
+      "  if (counter == 2)\n"
+      "    f.name = 42;\n"
+      "  else\n"
+      "    f.name = name + ':' + counter;\n"
+      "  return f;\n"
+      "};",
+      "origin");
+
+  v8::V8::AddMessageListener(StackTraceDisplayNameListener);

[yurys, 2015/03/05 14:12:56]

...DisplayName... -> ...FunctionName... here and below

[kozy, 2015/03/05 15:01:09]

Done.

+  v8::V8::SetCaptureStackTraceForUncaughtExceptions(true);
+  CompileRunWithOrigin("gen('foo', 2)();", "origin");
+  v8::V8::SetCaptureStackTraceForUncaughtExceptions(false);
+  v8::V8::RemoveMessageListeners(StackTraceDisplayNameListener);
+}
+
+
 static void RethrowStackTraceHandler(v8::Handle<v8::Message> message,
                                      v8::Handle<v8::Value> data) {
   // Use the frame where JavaScript is called from.