[V8] Use Function.name in Error.stack

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 7315496981c600dc910c6ec0b3a844db6a61e929..4814bcbcd45c95eb708924e2617214f45dece8e2 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -10254,6 +10254,16 @@ bool JSFunction::PassesFilter(const char* raw_filter) {
 }
 
 
+String* JSFunction::DebugName() {
+  HandleScope scope(GetIsolate());

[kozy, 2015/03/10 15:26:08]

Without this handle scope some tests fails with
# Fatal error in v8::HandleScope::CreateHandle()
# Cannot create a handle without a HandleScope
It seems strange for me, because we can't call this function without outer scope
in messages.js or CaptureStackTraceHelper.
Yang, please take a look on this line.

+  Handle<JSFunction> self(this);
+  Handle<Object> name =
+      JSObject::GetDataProperty(self, GetIsolate()->factory()->name_string());
+  if (name->IsString()) return String::cast(*name);
+  return shared()->DebugName();
+}
+
+
 void Oddball::Initialize(Isolate* isolate,
                          Handle<Oddball> oddball,
                          const char* to_string,