Reviewers: Yang,

Message:
ptal

Description:
messages.js: Get better function names in stack traces.

CallSite.getFunctionName() is able to retrieve names for functions better than
getFunction().name.  Use it in CallSite.toString().

BUG=NONE
TEST=NONE


Please review this at http://codereview.chromium.org/10384196/

SVN Base: git://github.com/v8/v8.git@master

Affected files:
  M src/messages.js


Index: src/messages.js
diff --git a/src/messages.js b/src/messages.js
index d813df9da6e26d035c02b4c72f5b7a4c2fc4baab..b95bc7a33645326fc6fbd753842f29a1d7efeac6 100644
--- a/src/messages.js
+++ b/src/messages.js
@@ -949,7 +949,7 @@ function CallSiteToString() {
     fileLocation = "unknown source";
   }
   var line = "";
-  var functionName = this.getFunction().name;
+  var functionName = this.getFunctionName();
   var addPrefix = true;
   var isConstructor = this.isConstructor();
   var isMethodCall = !(this.isToplevel() || isConstructor);


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to