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

test/mjsunit/error-constructors.js

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 30 matching lines @@
 var e0 = new ReferenceError();
 var e1 = new ReferenceError('123');
 assertTrue(e1.hasOwnProperty('message'));
 assertTrue(e0.hasOwnProperty('stack'));
 assertTrue(e1.hasOwnProperty('stack'));
 assertTrue(e0.hasOwnProperty('type'));
 assertTrue(e1.hasOwnProperty('type'));
 assertTrue(e0.hasOwnProperty('arguments'));
 assertTrue(e1.hasOwnProperty('arguments'));
 
+// Check that the name property on error prototypes is read-only and
+// dont-delete. This is not specified, but allowing overwriting the
+// name property with a getter can leaks error objects from different
+// script tags in the same context in a browser setting. We therefore
+// disallow changes to the name property on error objects.
+assertEquals("ReferenceError", ReferenceError.prototype.name);
+delete ReferenceError.prototype.name;

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

add
assertEquals("ReferenceError", ReferenceError.prototype.name);

just after delete as well?

+ReferenceError.prototype.name = "not a reference error";
+assertEquals("ReferenceError", ReferenceError.prototype.name);