AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Show function <name>() { [native code] } for built-in classes

Browse Source 
The existing logic would show the full source for all classes, even
built-in ones.

R=arv@chromium.org,dslomov@chromium.org
BUG=

Review URL: https://codereview.chromium.org/1112113002

Cr-Commit-Position: refs/heads/master@{#28151}
This commit is contained in:
domenic 2015-04-30 02:29:33 -07:00 committed by Commit bot
parent 4b122b7504
commit a34bbef314
1 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/v8natives.js
View File

@ -1738,6 +1738,16 @@ SetUpNumber();
// ----------------------------------------------------------------------------
// Function
function NativeCodeFunctionSourceString(func) {
var name = %FunctionGetName(func);
if (name) {
// Mimic what KJS does.
return 'function ' + name + '() { [native code] }';
}
return 'function () { [native code] }';
}
function FunctionSourceString(func) {
while (%IsJSFunctionProxy(func)) {
func = %GetCallTrap(func);
@ -1747,20 +1757,18 @@ function FunctionSourceString(func) {
throw MakeTypeError(kNotGeneric, 'Function.prototype.toString');
}
if (%FunctionIsBuiltin(func)) {
return NativeCodeFunctionSourceString(func);
}
var classSource = %ClassGetSourceCode(func);
if (IS_STRING(classSource)) {
return classSource;
}
var source = %FunctionGetSourceCode(func);
if (!IS_STRING(source) || %FunctionIsBuiltin(func)) {
var name = %FunctionGetName(func);
if (name) {
// Mimic what KJS does.
return 'function ' + name + '() { [native code] }';
} else {
return 'function () { [native code] }';
}
if (!IS_STRING(source)) {
return NativeCodeFunctionSourceString(func);
}
if (%FunctionIsArrow(func)) {