Added extra tests to the DefineOrRedefineAccessorProperty and ...

Added extra tests to the DefineOrRedefineAccessorProperty and 
DefineOrRedefineDataProperty to avoid invalid input.

Added tests to object-define-property.js to test that it does not crash 
on invalid input. 



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

[Mads Ager (chromium), 2010-02-04 15:08:42]

LGTM

http://codereview.chromium.org/572005/diff/1/3
File src/runtime.cc (right):

http://codereview.chromium.org/572005/diff/1/3#newcode2921
src/runtime.cc:2921: Handle<Object> name = args.at<Object>(1);
Use CONVERT_CHECKED(String, name, args[1]) here and remove the RUNTIME_ASSERT
and the Handle<String>::cast(name) from below.

[Rico, 2010-02-04 19:43:27]

http://codereview.chromium.org/572005/diff/1/3
File src/runtime.cc (right):

http://codereview.chromium.org/572005/diff/1/3#newcode2921
src/runtime.cc:2921: Handle<Object> name = args.at<Object>(1);
On 2010/02/04 15:08:42, Mads Ager wrote:
> Use CONVERT_CHECKED(String, name, args[1]) here and remove the RUNTIME_ASSERT
> and the Handle<String>::cast(name) from below.

Done.