Revision: 3279 Author: [email protected] Date: Wed Nov 11 03:28:37 2009 Log: Remove special-case for arguments.toString to match ES5
Patch by Jan de Mooij <[email protected]> Review: http://codereview.chromium.org/273073 Review URL: http://codereview.chromium.org/389008 http://code.google.com/p/v8/source/detail?r=3279 Modified: /branches/bleeding_edge/src/v8natives.js /branches/bleeding_edge/test/mjsunit/regress/regress-124.js ======================================= --- /branches/bleeding_edge/src/v8natives.js Thu Sep 17 04:11:01 2009 +++ /branches/bleeding_edge/src/v8natives.js Wed Nov 11 03:28:37 2009 @@ -196,10 +196,7 @@ // ECMA-262 - 15.2.4.2 function ObjectToString() { - var c = %_ClassOf(this); - // Hide Arguments from the outside. - if (c === 'Arguments') c = 'Object'; - return "[object " + c + "]"; + return "[object " + %_ClassOf(this) + "]"; } ======================================= --- /branches/bleeding_edge/test/mjsunit/regress/regress-124.js Fri Feb 27 05:00:32 2009 +++ /branches/bleeding_edge/test/mjsunit/regress/regress-124.js Wed Nov 11 03:28:37 2009 @@ -48,9 +48,9 @@ assertEquals("[object global]", eval("f()")); // Receiver should be the arguments object here. - assertEquals("[object Object]", eval("arguments[0]()")); + assertEquals("[object Arguments]", eval("arguments[0]()")); with (arguments) { - assertEquals("[object Object]", toString()); + assertEquals("[object Arguments]", toString()); } } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
