Revision: 9886
Author:   [email protected]
Date:     Mon Nov  7 01:30:28 2011
Log: Optimize DefineOneShotAccessor in messages.js to not call DefineOwnProperty.
Review URL: http://codereview.chromium.org/8490008
http://code.google.com/p/v8/source/detail?r=9886

Modified:
 /branches/bleeding_edge/src/messages.js

=======================================
--- /branches/bleeding_edge/src/messages.js     Fri Oct 28 01:45:04 2011
+++ /branches/bleeding_edge/src/messages.js     Mon Nov  7 01:30:28 2011
@@ -745,12 +745,8 @@
     hasBeenSet = true;
     value = v;
   }
-  var desc = { get: getter,
-               set: setter,
-               enumerable: false,
-               configurable: true };
-  desc = ToPropertyDescriptor(desc);
-  DefineOwnProperty(obj, name, desc, true);
+  %DefineOrRedefineAccessorProperty(obj, name, GETTER, getter, DONT_ENUM);
+  %DefineOrRedefineAccessorProperty(obj, name, SETTER, setter, DONT_ENUM);
 }

 function CallSite(receiver, fun, pos) {

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to