Make the 'name' property on error prototypes read-only and dont-delete...

Make the 'name' property on error prototypes read-only and dont-delete
to avoid leaking of error objects to accessor methods when formatting
error messages internally.

Also, do not call overwritten toString methods on error objects when
formatting messages internally.


Committed: http://code.google.com/p/v8/source/detail?r=6322

[Søren Thygesen Gjesse, 2011-01-14 12:47:26]

LGTM

http://codereview.chromium.org/6272004/diff/1/test/mjsunit/error-constructors.js
File test/mjsunit/error-constructors.js (right):

http://codereview.chromium.org/6272004/diff/1/test/mjsunit/error-constructors...
test/mjsunit/error-constructors.js:57: delete ReferenceError.prototype.name;
add
assertEquals("ReferenceError", ReferenceError.prototype.name);

just after delete as well?