Fix issue 965.

test/mjsunit/object-define-property.js

Index: test/mjsunit/object-define-property.js
diff --git a/test/mjsunit/object-define-property.js b/test/mjsunit/object-define-property.js
index b258aa75bf375e279e207746c383168d5ba62dd6..57d79b595f7b1171bdcaea7902e220b7d990ecd5 100644
--- a/test/mjsunit/object-define-property.js
+++ b/test/mjsunit/object-define-property.js
@@ -866,4 +866,15 @@ assertFalse(desc.writable);
 assertFalse(desc.enumerable);
 assertFalse(desc.configurable);
 
+// See issue 968: http://code.google.com/p/v8/issues/detail?id=968
+var o = { x : 42 };
+Object.defineProperty(o, "x", { writable: false });
+assertEquals(42, o.x);
+
+o = { x : 42 };
+Object.defineProperty(o, "x", {});
+assertEquals(42, o.x);
+o.x = 37;
+// Writability is preserved.
+assertEquals(37, o.x);
 

[Rico, 2010/12/10 10:42:35]

Maybe add an additional test that makes sure that an accessor is actually
overwritten correctly (step 9.c) if we only set the writable flag

[Lasse Reichstein, 2010/12/10 11:21:45]

More tests added, for both non-existing property, and for inherited value and
accessor properties,