Add a pretty printer to improve the error message non-function calls

src/runtime/runtime-debug.cc

 // Copyright 2014 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 "src/accessors.h"
 #include "src/arguments.h"
 #include "src/debug.h"
 #include "src/deoptimizer.h"
 #include "src/isolate-inl.h"
+#include "src/messages.h"
 #include "src/parser.h"
+#include "src/prettyprinter.h"
 #include "src/runtime/runtime.h"
 #include "src/runtime/runtime-utils.h"
 
 namespace v8 {
 namespace internal {
 
 RUNTIME_FUNCTION(Runtime_DebugBreak) {
   SealHandleScope shs(isolate);
   DCHECK(args.length() == 0);
   isolate->debug()->HandleDebugBreak();
@@ skipping 2697 matching lines @@
         break;
       }
     }
   }
 
   if (found.is_null()) return heap->undefined_value();
   return *Script::GetWrapper(found);
 }
 
 
+RUNTIME_FUNCTION(Runtime_RenderCallSite) {

[Yang, 2015/01/20 08:19:07]

Nit: runtime-debug.cc is used for debugger related stuff. This would belong to
runtime-internal.cc where the other stack trace and message related stuff are.

+  HandleScope scope(isolate);
+  DCHECK(args.length() == 0);
+  MessageLocation location;
+  isolate->ComputeLocation(&location);
+  if (location.start_pos() == -1) return isolate->heap()->empty_string();
+  CompilationInfoWithZone info(location.script());

[Yang, 2015/01/20 08:19:07]

If you could get the JSFunction from within ComputeLocation, and use it to
create the CompilationInfo, I'd imagine that parsing and ast visiting can go
quite a lot faster since you would only be creating AST for that particular
function.

+  if (!Parser::Parse(&info)) return isolate->heap()->empty_string();
+  Zone zone(isolate);
+  CallPrinter printer(&zone);
+  const char* string = printer.Print(info.function(), location.start_pos());
+  return *isolate->factory()->NewStringFromAsciiChecked(string);
+}
+
+
 // Check whether debugger and is about to step into the callback that is passed
 // to a built-in function such as Array.forEach.
 RUNTIME_FUNCTION(Runtime_DebugCallbackSupportsStepping) {
   DCHECK(args.length() == 1);
   Debug* debug = isolate->debug();
   if (!debug->is_active() || !debug->IsStepping() ||
       debug->last_step_action() != StepIn) {
     return isolate->heap()->false_value();
   }
   CONVERT_ARG_CHECKED(Object, callback, 0);
@@ skipping 69 matching lines @@
   return Smi::FromInt(isolate->debug()->is_active());
 }
 
 
 RUNTIME_FUNCTION(RuntimeReference_DebugBreakInOptimizedCode) {
   UNIMPLEMENTED();
   return NULL;
 }
 }
 }  // namespace v8::internal