Author: [email protected]
Date: Mon May 4 12:35:46 2009
New Revision: 1848
Removed:
branches/bleeding_edge/test/mjsunit/undeletable-functions.js
Modified:
branches/bleeding_edge/src/array.js
branches/bleeding_edge/src/date-delay.js
branches/bleeding_edge/src/math.js
branches/bleeding_edge/src/string.js
Log:
Reverty 1842 (see http://code.google.com/p/chromium/issues/detail?id=1717 )
Review URL: http://codereview.chromium.org/99346
Modified: branches/bleeding_edge/src/array.js
==============================================================================
--- branches/bleeding_edge/src/array.js (original)
+++ branches/bleeding_edge/src/array.js Mon May 4 12:35:46 2009
@@ -1011,10 +1011,9 @@
// object.
%SetProperty($Array.prototype, "constructor", $Array, DONT_ENUM);
- // Setup non-enumerable functions of the Array.prototype object and set
their
- // names. Use DONT_DELETE due to
- // http://code.google.com/p/chromium/issues/detail?id=1717
- InstallFunctions($Array.prototype, DONT_ENUM | DONT_DELETE, $Array(
+ // Setup non-enumerable functions of the Array.prototype object and
+ // set their names.
+ InstallFunctions($Array.prototype, DONT_ENUM, $Array(
"toString", ArrayToString,
"toLocaleString", ArrayToLocaleString,
"join", ArrayJoin,
Modified: branches/bleeding_edge/src/date-delay.js
==============================================================================
--- branches/bleeding_edge/src/date-delay.js (original)
+++ branches/bleeding_edge/src/date-delay.js Mon May 4 12:35:46 2009
@@ -1017,10 +1017,9 @@
// Setup non-enumerable constructor property of the Date prototype
object.
%SetProperty($Date.prototype, "constructor", $Date, DONT_ENUM);
- // Setup non-enumerable functions of the Date prototype object and set
their
- // names. Use DONT_DELETE due to
- // http://code.google.com/p/chromium/issues/detail?id=1717
- InstallFunctions($Date.prototype, DONT_ENUM | DONT_DELETE, $Array(
+ // Setup non-enumerable functions of the Date prototype object and
+ // set their names.
+ InstallFunctions($Date.prototype, DONT_ENUM, $Array(
"toString", DateToString,
"toDateString", DateToDateString,
"toTimeString", DateToTimeString,
Modified: branches/bleeding_edge/src/math.js
==============================================================================
--- branches/bleeding_edge/src/math.js (original)
+++ branches/bleeding_edge/src/math.js Mon May 4 12:35:46 2009
@@ -162,9 +162,9 @@
1.4142135623730951,
DONT_ENUM | DONT_DELETE | READ_ONLY);
- // Setup non-enumerable functions of the Math object and set their
names. Use
- // DONT_DELETE due to
http://code.google.com/p/chromium/issues/detail?id=1717
- InstallFunctions($Math, DONT_ENUM | DONT_DELETE, $Array(
+ // Setup non-enumerable functions of the Math object and
+ // set their names.
+ InstallFunctions($Math, DONT_ENUM, $Array(
"random", MathRandom,
"abs", MathAbs,
"acos", MathAcos,
Modified: branches/bleeding_edge/src/string.js
==============================================================================
--- branches/bleeding_edge/src/string.js (original)
+++ branches/bleeding_edge/src/string.js Mon May 4 12:35:46 2009
@@ -831,9 +831,7 @@
// Setup the non-enumerable functions on the String prototype object.
- // Use DONT_DELETE due to
- // http://code.google.com/p/chromium/issues/detail?id=1717
- InstallFunctions($String.prototype, DONT_ENUM | DONT_DELETE, $Array(
+ InstallFunctions($String.prototype, DONT_ENUM, $Array(
"valueOf", StringValueOf,
"toString", StringToString,
"charAt", StringCharAt,
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
