[runtime] [proxy] removing JSFunctionProxy and related code.

src/js/v8natives.js

 // Copyright 2012 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.
 
 (function(global, utils) {
 
 %CheckIsBootstrapping();
 
 // ----------------------------------------------------------------------------
 // Imports
@@ skipping 1524 matching lines @@
   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);
-  }
+  //TODO(cbruni): resolve func to a non-proxy function for printing

[Toon Verwaest, 2015/12/03 11:48:42]

The spec for Function.prototype.toString says only builtin functions and objects
with [[ECMAScriptCode]] are handled; so only real functions, not proxies. In all
other cases we throw a type error. Hence this comment isn't necessary.
IS_FUNCTION will return true for generators and jsfunctions (including builtins)
which is what we want.

[Camillo Bruni, 2015/12/03 12:18:26]

removed.

 
   if (!IS_FUNCTION(func)) {
     throw MakeTypeError(kNotGeneric, 'Function.prototype.toString');
   }
 
   if (%FunctionHidesSource(func)) {
     return NativeCodeFunctionSourceString(func);
   }
 
   var classSource = %ClassGetSourceCode(func);
@@ skipping 182 matching lines @@
 
 %InstallToContext([
   "global_eval_fun", GlobalEval,
   "object_value_of", ObjectValueOf,
   "object_to_string", ObjectToString,
   "object_get_own_property_descriptor", ObjectGetOwnPropertyDescriptor,
   "to_complete_property_descriptor", ToCompletePropertyDescriptor,
 ]);
 
 })